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
151 stars 62 forks source link

Support for workflow summary #89

Closed yijhenlin closed 3 weeks ago

yijhenlin commented 6 months ago

Add an option for workflow summary. Fix #88

thsaravana commented 4 weeks ago

Sorry about the late reply. We just picking this for milestone v1.7.2. The dist is auto-generated files. Could you make the changes to the .ts files with this change?

thsaravana commented 3 weeks ago

We are taking a different approach to this:

  1. We will have an option to either do a PR Comment or Workflow Summary, as well as "both" at the same time. We will develop this further on adding more detailed report in the workflow summary later.
  2. The token can't be optional, since we are using this to calculate the "changed files". So we are going to keep it required

We would still like to give you credit for raising the issue and raising the initial PR. Thanks!