-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Coverage Issue for HashTable and TimSorter Reducing Overall Code Coverage #465
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
Comments
Thank you for opening the issue, it probably happens because of randomization in tests, so coverage fluctuates a bit. I tried fixing some places where this happened, but never got to all of them. Just to make it clear, you are working on a fix, right? I'll assign you to the issue then |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
seems this bug fixed with PR 486 |
#486 didn't change the classes in question or their tests, so I think the issue is still present |
Hi siriak is this issue still opened, an you assigned it to me? I think I can work on it. I could rescue my account back. |
is any one still working on this? |
Uh oh!
There was an error while loading. Please reload this page.
Summary
We’ve encountered a coverage issue related to two classes, HashTable and TimSorter, which is impacting the overall code coverage in our project. Despite the current PR achieving 100% coverage for the changes it introduces, these two classes are not meeting the coverage standards, which is causing a blockage due to Codecov warnings.
To Reproduce
Expected behavior
Actual behavior
The text was updated successfully, but these errors were encountered: