Skip to content

Add optional support for diff_cover reports#2

Open
jorgeAtViboo wants to merge 18 commits intomainfrom
prettify
Open

Add optional support for diff_cover reports#2
jorgeAtViboo wants to merge 18 commits intomainfrom
prettify

Conversation

@jorgeAtViboo
Copy link
Owner

@jorgeAtViboo jorgeAtViboo commented Mar 4, 2025

  • Adds an optional argument diffCoverageFile to the action which is a report from https://github.com/Bachmann1234/diff_cover which offers a more detailed list of line changed in the diff file as opposed to the modified file in it's entirety. The goal being to only evaluate the changes made and not get punished for making changes in a large untested file.
  • Built files are also included in the original repo and serve for the test.yml action. You can forget about them for the review.
  • The comment below is generated by said action with the base being the example coverage files coverage-diff.json and coverage.xml.

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1000 940 94% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/coverage.ts 69% 🟢
TOTAL 69% 🟢

updated for commit: ff6ed38 by action🐍

@jorgeAtViboo jorgeAtViboo changed the title Improve log message formatting for clarity in commit comparison Add optional support for diff_cover reports Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants