Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to skip reporting when no files are changed #53

Open
wants to merge 36 commits into
base: master
Choose a base branch
from

Conversation

andybelltree
Copy link

@andybelltree andybelltree commented Aug 29, 2022

  1. Added an option dont-post-if-no-changed-files-in-report. If this is set to true, and a report is generated with no changed files, it doesn't post a comment. This is to reduce noise (our use case is a repo with a number of projects, and we don't want a report posted about every project for every change, but we still do want the tests to run for downstream projects)
  2. If an empty report is generated without this option, and no files are changed, (and filter-changed-files is set) then don't publish an empty table with a header
  3. Added max-uncovered-lines option. If supplied will limit the number of uncovered lines displayed in the coverage column.
  4. Added fail-drop-percent-threshold. If supplied, and doing a diff against an existing report, the gha will fail when percentage change is greater than given value.
    Can test it out with andybelltree/[email protected]

andybelltree and others added 24 commits November 15, 2021 16:16
Fix boolean parameters being passed in as strings
… to true, and no changed files for the PR would be included in the report, then skip posting.
Add an option dont-post-if-no-changed-files-in-report. If this is set…
Move substring after checking for undefined
Remove summary title too if the table is empty.
…rage

Add a failure threshold for code coverage dropping
klausbadelt added a commit to klausbadelt/lcov-reporter-action that referenced this pull request Mar 5, 2023
* max-uncovered-lines
* dont-post-if-no-changed-files-in-report
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