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

Check coverage with borp? #23

Closed
Palaxx opened this issue Apr 3, 2024 · 0 comments · Fixed by #25
Closed

Check coverage with borp? #23

Palaxx opened this issue Apr 3, 2024 · 0 comments · Fixed by #25

Comments

@Palaxx
Copy link
Contributor

Palaxx commented Apr 3, 2024

Hi,

I am trying to find out whether it is possible to check coverage via borp.
Looking at the code I understand that it is currently not yet possible.

I've tried to imagine a method to enable the check via PR, and I'm guessing that the following things need to be done:

  • New parameter to enable coverage --check-coverage
  • New parameter to define tresholds ( --lines, --functions, etc)
  • Running the c8 checkCoverages command after the reporter run.

If we did not want to expose the new parameters we could perhaps tie the PR to this #17 and have the parameters retrieved from the c8 conf file.

Do you think this is an interesting feature to introduce in borp?
Could the proposed solution be a valid approach?

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 a pull request may close this issue.

1 participant