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.

Sign up for GitHub

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

Jump to bottom

PR comment when no JaCoCo report is found #91

Closed
ivanmarban opened this issue Apr 26, 2024 · 0 comments · Fixed by #128
Closed

PR comment when no JaCoCo report is found #91

ivanmarban opened this issue Apr 26, 2024 · 0 comments · Fixed by #128
Assignees
@thsaravana
Milestone
1.7.1

Comments

@ivanmarban
Copy link

Would be nice to replace the PR comment when action does not find the reports. Actually it looks like this if there is no report present in path:

Code Coverage

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

Maybe a generic message on the PR warning you that report couldn't be found...
@thsaravana thsaravana self-assigned this Aug 30, 2024
@instrap instrap added this to the 1.7.1 milestone Aug 31, 2024
@thsaravana thsaravana mentioned this issue Sep 8, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thsaravana thsaravana

Labels
None yet
Projects
None yet
Milestone
1.7.1
Development

Successfully merging a pull request may close this issue.

Handle coverage when report is empty Madrapps/jacoco-report
3 participants
@ivanmarban @thsaravana @instrap