CI coverage test #1794
CI coverage test #1794
Conversation
Coverage report (presidio-anonymizer)
This PR does not seem to contain any modification to coverable code. |
Coverage report (presidio-structured)
This PR does not seem to contain any modification to coverable code. |
Coverage report (presidio-cli)
This PR does not seem to contain any modification to coverable code. |
…d improve output formatting
📊 Coverage Report
Threshold: 70% minimum |
…y and improve formatting
📊 Coverage Report
This shows total coverage for each component. |
…and enhance PR comment functionality
…r merging and updating upload paths
…ation file and simplify coverage file handling
…on PRs, removing combined coverage report steps
…TML report and improve file change detection logic
…conditions and improve coverage output handling
… coverage and upload coverage metrics
📊 Coverage Report
Total Coverage: All code in component Threshold: 70% minimum |
📊 Coverage Report
Total Coverage: All code in component Threshold: 70% minimum |
Coverage report (Anonymizer)
This PR does not seem to contain any modification to coverable code. |
Coverage report (CLI)
This PR does not seem to contain any modification to coverable code. |
Coverage report (Structured)
This PR does not seem to contain any modification to coverable code. |
Coverage report (Image Redactor)
This PR does not seem to contain any modification to coverable code. |
…culate_pii_density method
Coverage report (Analyzer)
This PR does not seem to contain any modification to coverable code. |
…s and primary Python in tests
…efine coverage threshold
Change Description
This PR introduces two improvements to our CI coverage workflow:
Each pipeline run posts a comment that shows the coverage of the specific component being tested.
The action updates the existing comment instead of creating new ones on every run.
Example:
We now verify coverage only for the lines changed in the PR.
If the PR reduces diff-coverage below 80%, the CI step fails.
Example of failure:
Additionally, a COVERAGE_THRESHOLD env variable was added so the required percentage can be easily raised in the future.