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
Enforce Plone 5.2/Python 3 compatibility #98
Conversation
996ccb1
to
c6b6161
Compare
This acquisition wrapper manually wraps an instance of an object, but tests seems to be overkill and the method seems to be removed from Zope 4.
@jensens can you take a look on this? there's a small encoding issue with one test but in general I think the package must be compatible with Plone 5.2 and Python 3. |
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.
Looks good, expect the encoding problem, which need to be solved.
@jensens any idea on how to solve it? |
@hvelarde I think I fixed it. Merge if green? |
refs. #97