-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Increase format presision of Code to Test Ratio #230
Comments
When |
Thank you for your report!
Is this Issue about
Is it a decimal point? Or is it a problem that is not displayed? |
Perhaps octocov (or octocov-action) is not getting coverage at this time. One way to debug is to download the coverage data when octocov fails to get coverage and run the octocov command locally. |
It takes a lot of time and effort to increase the test coverage by 10%. Sometimes it is impossible to achieve it, for example from 80% to 90% and from 90 to 100.
Why don't we use 2 or 3 decimal places as default? In my case, this change will motivate me to write test codes.
This may break changes but I think it won't change any system behaviors.
If you agree the proposal, I'll make a pull-request👍
https://github.com/k1LoW/octocov/blob/bdda036c59ac40b949f6b76f6e4f741e2606763a/report/report.go#LL111C32-L111C36
The text was updated successfully, but these errors were encountered: