-
Notifications
You must be signed in to change notification settings - Fork 153
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
remove pillow dependency #1897
remove pillow dependency #1897
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. @cadecairos do you also want to take a look?
@Pomax can you rebase and fix the conflict so we can merge this? |
I can't really get the pip lock file to only update the pillow part and the master hash, a new venv ends up bumping some things automatically. |
Aparently, no rebasing/respinning I can do on my windows machine seems to fix this piplock issue. |
Redid the PR on current master. |
@cadecairos just reset you for review, just to make sure we had eyes on it before merge =) |
While wagtail relies on pillow, we don't explicitly use it ourselves, and so we don't need it in our our pipfile