[experimental] Dev docker fixes dom2022 #3949
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- change from 'collections' to 'collections.abc' - later python versions changed where this was imported from
ended up not needing these as the errors I was working around were config issues, but, not a bad idea to keep them anyway.
- improves confusion on first connection attempt - all this code here can use a rethink
- upgrade to python 3.10 - update many packages for python 3.10 compatibility - rip apart setup.py for uber and ubersystem. I for SURE broke stuff here. - change pavement.py to explicitly install plugin dependencies - remove NPM install stuff from Dockerfile - implement multi-stage dockerfile support for dev vs prod (to mount your own code in the container for easier dev support) - this doesn't quite get a running app, but, it gets as far as cherrypy starting - DO NOT USE THIS without a TON more testing - does allow debugging in PyCharm :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes Docker, warning: EXPERIMENTAL UNFINISHED
see magfest/sideboard#108 for full writeup, the most important changes are there.
most of the stuff in this PR is for upgrading to python 3.10.x