-
Notifications
You must be signed in to change notification settings - Fork 78
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
remove redundant .coveragerc
#604
Conversation
Codecov Report
@@ Coverage Diff @@
## main #604 +/- ##
=======================================
+ Coverage 83.9% 84.1% +0.2%
=======================================
Files 127 137 +10
Lines 14804 14781 -23
=======================================
+ Hits 12420 12435 +15
+ Misses 2384 2346 -38 |
The contents we have in |
Good question. The contents in
Is there anything else you'd like to see in Also, I wonder if there is any reason not to be using |
Perhaps this has something to do with it?
|
Pretty significant runtime decrease in some, negligible in others. Any thoughts on the |
I have removed it recently precisely for the file access conflicts. We can try to resolve those by designing the test differently but let's deal with it separately if you feel this will significantly help. |
not really, I was just wondering why the difference. |
This PR resolves #602.