Skip to content
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

Fix incompatibility with Pillow 10.1+. #90

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Conversation

mauritsvanrees
Copy link
Member

Instead of setting image.mode, we call image.convert. This works in Pillow 9 and 10.
Fixes #89.

Instead of setting ``image.mode``, we call ``image.convert``. This works in Pillow 9 and 10.
@mister-roboto

This comment was marked as resolved.

@mauritsvanrees
Copy link
Member Author

@jenkins-plone-org please run jobs

@mauritsvanrees mauritsvanrees merged commit 5f7f340 into master Jan 29, 2024
13 checks passed
@mauritsvanrees mauritsvanrees deleted the maurits-pillow-10 branch January 29, 2024 08:59
mauritsvanrees added a commit to plone/buildout.coredev that referenced this pull request Jan 29, 2024
This should work, now that plone/plone.scale#90 is merged.

I am inclined to say we only do this for Plone 6.1, not 6.0.  The release has backwards incompatibilities.  Should not be noticeable under normal use, but you never know what projects have as extra code.
I am not aware of anything new in Pillow 10 that we would really want to actively use in Plone.  So 9.5 should still be fine for Plone 6.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incompatible with Pillow 10.1+
3 participants