-
Notifications
You must be signed in to change notification settings - Fork 0
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
looks cool ✨ #7
Comments
That would be great, @gingerlime. How should we proceed? What you think is the best option? |
@scorphus cool. Would you be comfortable opening a PR to add wand-engine into the Docker image? are you comfortable with docker to test it? I can definitely help with adding it to the baseline, but most likely need your help testing that things are working :) |
Definitely.
Comfortable enough, I'd say.
Awesome! Is there any change made in the past (one or more commits in git history) that may be helpful in some ways? |
Hey @scorphus sorry for the delay. The main thumbor Dockerfile should be the best starting point I assume? plus the requirements.txt file. In addition, the thumbor configuration lives in a template file that gets parsed to set up the actual config... Let me know if you have any other questions or need help with anything else :) |
Hey, @gingerlime. Sorry for the delay too. I gave it a try and it seemed to me that the container runs Python 2. The engine was born after the retirement of Python 2 and I wouldn't like to add support to retired software. Things should move forward 🙂 I then managed to install Thumbor@master ― with Would you be willing to add support to Python 3 and Thumbor@master as some sort of alpha image release? |
Yeah, thumbor seems to move a bit slow to python 3 if I'm not mistaken? haven't seen any announcement or much changing, but I'm not following very closely. Definitely keen to get things over to python 3. Take a look at MinimalCompact/thumbor#76 where I started some work in this direction, but it's been kinda quiet on that one for a while ... This should be a good baseline though I think. Please feel free to comment there or ask on Thumbor to see where things are going. |
Hey, @gingerlime 👋 I'm almost certain you're already aware of thumbor 7 being released recently. I just felt like giving a heads up 🙂 |
I'm well aware of it :) |
Hey, @gingerlime! What do we do about this issue? Should we transfer it to where changes have to be made to? |
This looks cool. I'm maintaining the thumbor docker image at https://github.com/minimalcompact/thumbor
Would you be interested creating a PR to integrate thumbor-wand-engine into the docker image? happy to help in the process. I would say this should be strictly optional, but could increase adoption if it's available out of the box (or container) ;)
The text was updated successfully, but these errors were encountered: