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

[Docker] Bring back python2 dep #666

Merged
merged 1 commit into from
Jan 26, 2022
Merged

Conversation

lchrusciel
Copy link
Member

Revert of #643. It is still needed due to node and gyp dependency

@lchrusciel lchrusciel requested a review from a team as a code owner January 25, 2022 10:48
@lchrusciel lchrusciel added Docker Docker-related issues and PRs. Bug labels Jan 26, 2022
@Zales0123 Zales0123 merged commit 0940aa5 into Sylius:master Jan 26, 2022
@Zales0123
Copy link
Member

Thanks, Łukasz! 🎉

@lchrusciel lchrusciel deleted the docker-python branch January 26, 2022 10:37
@Ferror
Copy link
Contributor

Ferror commented Feb 15, 2022

@lchrusciel I would kindly recommend creating a pipeline that could build the docker image and make sure that at least it builds properly 😢 I see that this topic comes back like a boomerang without any proper comments.

@lchrusciel
Copy link
Member Author

This time I at least mention, that it is required by node dependency ;) Other than that, of course! If you, however, would like to contribute something (may be just review or thoughts) please join our slack and write it on #docker channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Docker Docker-related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants