-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add support for Python 3.6 #10
Conversation
Fixes #9 Also update to current tox.ini and .travis.yml to enable coverage.
@@ -1 +1 @@ | |||
__import__('pkg_resources').declare_namespace(__name__) | |||
__import__('pkg_resources').declare_namespace(__name__) # pragma: no cover |
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.
That one is odd: I'm guessing it gets skipped entirely by the namespace machinery, but then why does coverage
pick it up at all?
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'm guessing it gets skipped entirely by the namespace machinery
Right, as far as imports go.
why does coverage pick it up at all?
Because the report phase is based on the files it finds on disk, I'm assuming. It's the same for any module that's not imported at all but is still on disk.
src/zope/deprecation/tests.py
Outdated
def _makeOne(self, msg, *args): | ||
cls = self._getTargetClass() | ||
return cls(msg, *args) | ||
|
||
def fixture(self): | ||
return 'fixture' | ||
raise NotImplementedError() |
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 think this method could just be removed altogether.
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.
Or we could (did) add a test case that calls it :)
Thank you! |
Fixes #9
Also update to current tox.ini and .travis.yml to enable coverage.