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

Rounding? #17

Closed
Twipped opened this issue Mar 9, 2023 · 3 comments · Fixed by #20
Closed

Rounding? #17

Twipped opened this issue Mar 9, 2023 · 3 comments · Fixed by #20
Labels
enhancement New feature or request

Comments

@Twipped
Copy link

Twipped commented Mar 9, 2023

I'd like to have it only bump coverage numbers down to the nearest whole value. Is there an option for this that I'm not seeing? Threshold appears to still set it to the thousandths place once you cross the threshold.

We're having a lot of issues with merge conflicts on the jest config because the coverage values change for every single PR.

@rbardini
Copy link
Owner

There's no option for rounding. Maybe a reasonable default (e.g. two digits) would be enough?

@aschillio
Copy link

I have the exact same issue, any workaround on this ?

@rbardini
Copy link
Owner

rbardini commented Jul 3, 2023

A workaround is to define a higher margin, so that thresholds are only increased when they go above that value.

@rbardini rbardini added the enhancement New feature or request label Jul 3, 2023
@rbardini rbardini linked a pull request Dec 28, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants