-
Notifications
You must be signed in to change notification settings - Fork 55
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, Plone 5.2 and Plone 6 #900
Conversation
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, guys, but it's not possible for me to review a PR with 48 commits and 106 files changed
This is ready for merging. |
lazy-object-proxy > 1.6.0 is not compatible with Python 2
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 reviewed and left some comments.
The default Python 3 isort rules are different from the Python 2 isort rules. So we need to ignore I004 in Python 2.
Use Products.CMFPlone.utils.safe_text instead of Products.CMFPlone.utils.safe_unicode. safe_unicode is deprecated.
Plone already install it.
We already have part rebuild_i18n-sh in buildout.
Tests in Plone 5.2 doesn't install the product into portal_quickinstaller. So, to make reinstall in portal_quickinstaller, we need to install the product in portal_quickinstaller.
Since we don't use RequireJS, compile must be false. Also renames the configuration file to bootstrapjs.xml.
So, when we go to fix the robot tests, this will already be working
Avoid error: Module build failed: BrowserslistError: Unknown browser query `dead` Ref: browserslist/browserslist#266 (comment)
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.
Some adjustments still need to be made but can be done in other PRs so this one doesn't get even bigger.
No description provided.