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
No Pull Request comment added on push #62
Comments
The problem seems to be here: action.ts
If the prNumber is undefined we don't add a comment. In the case of a "push" there is no prNumber set. I've managed to get this working by getting the PR number associated with the commit.
Overall this means pushing changes will not update the code coverage comment without this change (or similar). |
@thsaravana |
I am already making some changes in 1.7 in this area. So I will test this as well and will fix this in 1.7 properly. |
isn't the workaround also to trigger the workflow not on |
The problem with that is that you always want to build and test on push, at least in main. Adding a comment to an associated PR is an optional addition. I wouldn't want to rebuild/retest everything just for the coverage comment. |
Is there still any progress on this? |
I've been having trouble trying to get the plugin to add a comment to the PR. I've enabled
debug-mode
and it successfully finds the Jacoco xml file. It outputs messages about the changes but doesn't mention trying to add a comment. The action is successful.Logs
I can successfully add comments myself using a separate github action so I don't believe this is permission related.
Adds a comment successfully
Am I using this incorrectly? Should it fail if it doesn't add a comment?
The text was updated successfully, but these errors were encountered: