-
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
Chore: devcontainer cleanups, update python version #34
Conversation
e3539ae
to
8650966
Compare
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.
These changes look good to me!
Would it make sense to add multi-platform support to this template devcontainer? Like what was done in cal-itp/docker-python-web #36
easier to interact with Docker Compose this way
make python version a build arg
3.11 is the default
better support for engineers across Windows, Mac (incl Apple Silicon), and Linux
bab5c16
to
6008fc1
Compare
Great suggestion / catch! This would have been annoying for @lalver1 and @machikoyasuda trying to use this devcontainer! Added in |
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.
🚀
Looks good to me too! And thanks for the multi-platform support 😄 |
🟢 I ran the workflow manually here: https://github.com/compilerla/template-devcontainer/actions/runs/9912876071
The newly tagged version images were built and pushed to GHCR:
main
(now Python 3.11)main_3.10
main_3.12