-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use Dockerfile to build image #1382
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.
Caught a small typo. Thanks for the doing this to avoid breakages!
justfile
Outdated
@@ -34,6 +34,9 @@ _uid_args := if os_family() == "unix" { | |||
"" | |||
} | |||
|
|||
PYTHON_VESION := env("PYTHON_VESION", "3.12.1") |
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.
PYTHON_VESION := env("PYTHON_VESION", "3.12.1") | |
PYTHON_VERSION := env("PYTHON_VERSION", "3.12.1") |
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.
Thanks! I'll fix it.
Intent
CI was broken by an upstream change to the docker container we rely on. This PR allows us to control the versions of Python and Quarto we want to use so we won't be broken by future changes.
Type of Change
Approach
Automated Tests
Directions for Reviewers
Checklist