search

Madrapps / jacoco-report

Github action that publishes the JaCoCo report as a comment in the Pull Request
https://github.com/marketplace/actions/jacoco-report
MIT License
146 stars 63 forks source link

Show full coverage report on new / modified files #38

Closed pavelfomin closed 1 week ago

pavelfomin commented 1 year ago

Thank you for making this action available! One suggestion I have is to include full coverage report for new / modified files. Typically, to approve a PR I would like to see the full coverage report on new / modified files (w/out checking out the branch and running the build to generate the full code coverage report locally):

image

Would be nice to have this feature.

thsaravana commented 1 year ago

It's not possible to add the full jacoco report to the PR comment. That would be really long. May be, we can expose this report in the Github workflow run. Let me see if we can do that!

MichaelMiles commented 5 months ago

yeah it would be nice to have a full jacoco report shown somewhere, is this feature in progress?

instrap commented 1 week ago

I took a look at this, and this doesn't seem to be feasible. The easiest option would be to just add another step in your workflow to upload the Coverage html package. Trying to implement what jacoco does is a huge venture. We are working on a couple of things that will enable this partially. #88 #60 #67 . But there is no plans to upload the full html as part of this action. So closing this.