Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comment is updated even with option in false #21

Open
ghost opened this issue Jul 21, 2021 · 5 comments · May be fixed by #25
Open

Comment is updated even with option in false #21

ghost opened this issue Jul 21, 2021 · 5 comments · May be fixed by #25

Comments

@ghost
Copy link

ghost commented Jul 21, 2021

I have a workflow with two different jobs which each of them has this GA action.

 - name: Publish code coverage report
    uses: vebr/jest-lcov-reporter@v0.2.1
     with:
        github-token: ${{ github.token }}
        name: Client Portal Coverage Report
        lcov-file: webapps/client-portal/coverage/lcov.info
        update-comment: false
- name: Publish code coverage report
    uses: vebr/jest-lcov-reporter@v0.2.1
    with:
       github-token: ${{ github.token }}
       name: Qualifications Portal Coverage Report
        lcov-file: webapps/qualifications-portal/coverage/lcov.info
        update-comment: false

The issue is always there is only one comment created and it's being updated, so I never have the two comments.

@ricardovanlaarhoven
Copy link

Warning: Unexpected input(s) 'update-comment', valid inputs are ['github-token', 'lcov-file', 'lcov-base']

@ricardovanlaarhoven
Copy link

The latest release github action failed, that release included this feature. Please take a look into that.
i'd love to help but i cant check the logs or run the command

@alexjfno1
Copy link
Contributor

The issue is always there is only one comment created and it's being updated, so I never have the two comments.

At the moment, if update-comment is enabled, it will only leave one comment per workflow as it uses the workflow name as the unique identifier. The feature would need to be enhanced if you wanted to update more that one comment per workflow.

As for the update-comment: false, I will see if I can reproduce the issue.

@alexjfno1
Copy link
Contributor

alexjfno1 commented Oct 5, 2021

@ricardovanlaarhoven Initial investigations look like the getInput only returns string values - actions/toolkit#361 which means that checking the input value returns "false" which evaluates to be true.

We need to use the new getBooleanInput - actions/toolkit#725

I will try to get a PR raised 😃

alexjfno1 added a commit to alexjfno1/jest-lcov-reporter that referenced this issue Oct 5, 2021
@alexjfno1 alexjfno1 linked a pull request Oct 5, 2021 that will close this issue
@MishaKav
Copy link

I find the solution in a similar package
https://github.com/marketplace/actions/jest-coverage-comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants