-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
MAINT moved all "d" modules to pytest. #7885
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
Conversation
|
I removed the corresponding commit. |
@@ -1,12 +1,12 @@ | |||
from __future__ import (absolute_import, division, print_function, | |||
unicode_literals) | |||
|
|||
import six |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The line below uses six.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As an import?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see where the import is useful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I'm not able to find any use of six.
in that file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I wasn't thinking properly.
Current coverage is 62.17% (diff: 100%)@@ master #7885 diff @@
==========================================
Files 174 174
Lines 56121 56121
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 34894 34896 +2
+ Misses 21227 21225 -2
Partials 0 0
|
Used Github's UI to resolve a merge conflict that came from merging the test_c* PR before this one. I'll let the CI run and then finish the merge. |
This PR migrates the modules starting by "d" from nosetests to pytest.