Skip to content

Conversation

@drone-ah
Copy link
Contributor

@drone-ah drone-ah commented Feb 6, 2025

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,

@zgosalvez
Copy link
Owner

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

@drone-ah
Copy link
Contributor Author

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?

@KirienzoEth
Copy link

KirienzoEth commented Sep 25, 2025

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!

@zgosalvez zgosalvez merged commit a4dc6cc into zgosalvez:main Sep 27, 2025
1 of 2 checks passed
@zgosalvez
Copy link
Owner

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

@drone-ah drone-ah deleted the genhtml-args branch October 6, 2025 12:35
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.

3 participants