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
Python 3 fixes and tests #34
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.
lgtm
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.
Apart from two change requests this LGTM!
- Please check my comment in filtercomments.py
- Please remove the 5.0 compatibility introduced here and keep what is useful (e.g. we don't need to check for 5.0 anymore, but letting portlets derive from
base.AddForm
is cool).
Make compatible with 5.0.x #25
I released a 2.1 version and rebased your changes on top of that. Since we got 5.0 support, which is removed with this PR, this will become a 3.0 release.
awesome work, thanx! |
No description provided.