Add coverage report as new step in CI #117
Conversation
|
Ready. Now coverage is reported in every pull request under the Actions tab.
|
|
Still I think the coverage report should be posted automatically in a comment instead of the Actions page. |
There was a problem hiding this comment.
How does this keep the CI up to date?
There was a problem hiding this comment.
It inspects the CI actions regularly checking if there are new versions. If it finds it, the dependantbot creates a PR with the version bump. If you agree with the change, just merge it.
There was a problem hiding this comment.
see e.g. cta-observatory/magic-cta-pipe#198
There was a problem hiding this comment.
Related to this I'm actually seeing that in the Security tab, there is this same information https://github.com/juanep97/iop4/security/dependabot. Some warnings on packages that should be updated
Let's do it here. I will explore the two options (an already existing action that upload the coverage report as a comment or use GH action script to create the comment manually) |
|
Would something like https://github.com/orgs/community/discussions/69944#discussioncomment-7269357 work? |
Could be, to be tried. Maybe we can generate the table in markdown format as shown above and post it as a comment. The problem with the action I just tried |
closes #45