-
Notifications
You must be signed in to change notification settings - Fork 3
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
Port to Python 3 #1
Conversation
94e04c3
to
05da517
Compare
The tests pass, but they need unreleased versions of zope.app.basicskin (zopefoundation/zope.app.basicskin#1) and zope.app.pagetemplate (zopefoundation/zope.app.pagetemplate#1). We also have a workaround for zopefoundation/zope.mimetype#6 (no issue/fix yet) and the unreleased zopefoundation/zope.publisher#13
setup.py
Outdated
'License :: OSI Approved :: Zope Public License', | ||
'Programming Language :: Python', | ||
'Programming Language :: Python:: 2.7', | ||
'Programming Language :: Python:: 3.5', |
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.
Python 3.4 support, please.
""" | ||
from __future__ import print_function, absolute_import, division |
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.
Not needed.
@@ -15,9 +15,10 @@ | |||
|
|||
$Id$ | |||
""" | |||
from __future__ import print_function, absolute_import, division |
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.
Not needed.
""" | ||
Tests for deprecated/backwards-compatibility imports. | ||
""" | ||
from __future__ import print_function, absolute_import, division |
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.
Not needed.
""" | ||
from __future__ import print_function, absolute_import, division |
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.
Not needed.
>>> api = ZopeTalesAPI(testObject) | ||
>>> api.created | ||
datetime.datetime(2000, 10, 1, 23, 11) | ||
""" |
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.
Yay!
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.
There were few enough doctests here that I was able to get away with not using a RENormalizer
and just convert to unit tests 😄
""" | ||
|
||
from __future__ import absolute_import, print_function, division |
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.
Not needed.
""" | ||
from __future__ import print_function, absolute_import, division |
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.
Not needed.
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.
lgtm
.travis.yml
Outdated
script: | ||
- python setup.py test -q | ||
- coverage run -m zope.testrunner --test-path=src --auto-color --auto-progress |
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.
My inner pedant cannot not notice the indentation change here.
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.
My bad. Corrected.
Thanks! A PyPI release would be handy when convenient. |
@jamadden what's your PyPI username? |
@mgedmin It's... jamadden. I'm predictable. Mostly 😉 |
You now have PyPI rights to zope.app.pagetemplate. Congratulations! |
Cool, thanks. If there are no objections, I'll go ahead and release this soon. |
No description provided.