github-actions-report-lcov icon indicating copy to clipboard operation
github-actions-report-lcov copied to clipboard

allow setting errors to ignore for genhtml

Open drone-ah opened this issue 9 months ago • 2 comments

Hi,

This PR allows setting the --ignore-errors flag, mainly for the empty one which is handy when first setting up bdd tests and no code has been written yet.

Please let me know if it requires any further changes to be accepted.

Thanks,

drone-ah avatar Feb 06 '25 12:02 drone-ah

I'd like to release this under v4.2. Would you mind checking with #293 so we can have both PRs released?

zgosalvez avatar Feb 09 '25 06:02 zgosalvez

I'd like to release this under v4.2. Would you mind checking with #293 so we can have both PRs released?

I'd be happy to rebase and resolve any merge conflicts once #293 is merged. I'll also update the documentation.

One question I have is whether the parameter should be renamed to just ignore-errors - what do you think?

drone-ah avatar Feb 12 '25 09:02 drone-ah

Hello @zgosalvez @drone-ah

Do you know when this can be merged and included in a new version? Our testing framework generates inconsistent linecov/branchcov lcov files and now the workflow blocks PRs, so we had no other choice than to disable it temporarily.

I'm down to resolve conflicts and review both #292 and #293 if that can help.

Thank you for your work!

KirienzoEth avatar Sep 25 '25 06:09 KirienzoEth

I appreciate the nudge guys! Merging this along with other housekeeping PRs to v5.0.0. Thank you for your patience and contribution! 🙏🏼

zgosalvez avatar Sep 27 '25 06:09 zgosalvez