-
Notifications
You must be signed in to change notification settings - Fork 52
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
feat(deps): Add devbox
config
#2914
Conversation
b735d64
to
4a92a0f
Compare
# Issue Specifying exact versions for dependencies in Nix is not straightforward. # Fix This commit adds a basic `devbox` configuration for our project. If you want to try it out: [Install devbox] and run `devbox shell` [Install devbox]: https://www.jetpack.io/devbox/docs/installing_devbox/
# Issue The required version of the concourse CLI is dictated by the server. concourse.app-runtime-interfaces.ci.cloudfoundry.org is on 7.10.0 # Fix Rollback and disable automated updates through renovate
Quality Gate passedIssues Measures |
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.
I think it is better to test first, if the devbox-binary is available and only then do that eval. Similarly as it has been before.
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.
Good point: #2921
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.
Is that file needed here for devbox? Can we avoid double-maintenance of those two packages?
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.
Yes, right. I hope it is possible to reuse this local-flake from the top-level flake?
Maybe you know how? 😄
If not I'll check.
# Issue Specifying exact versions for dependencies in Nix is not straightforward. # Fix This commit adds a basic `devbox` configuration for our project. If you want to try it out: [Install devbox] and run `devbox shell` [Install devbox]: https://www.jetpack.io/devbox/docs/installing_devbox/ * fix(deps): Rollback `concourse` version # Issue The required version of the concourse CLI is dictated by the server. concourse.app-runtime-interfaces.ci.cloudfoundry.org is on 7.10.0 # Fix Rollback and disable automated updates through renovate * 🤖🦾🛠️ scripts/asdf2devbox.py
Specifying exact versions for dependencies in Nix is not straightforward. This commit adds a basic `devbox` configuration for our project. If you want to try it out: [Install devbox] and run `devbox shell` [Install devbox]: https://www.jetpack.io/devbox/docs/installing_devbox/ * fix(deps): Rollback `concourse` version The required version of the concourse CLI is dictated by the server. concourse.app-runtime-interfaces.ci.cloudfoundry.org is on 7.10.0 Rollback and disable automated updates through renovate * 🤖🦾🛠️ scripts/asdf2devbox.py
Issue
Specifying exact versions for dependencies in Nix is not
straightforward.
Fix
This commit adds a basic
devbox
configuration for our project.If you want to try it out: Install devbox and run
devbox shell
This PR also adds a script that tries to keep the version in
devbox.json
in sync with those in.tool-versions
,scripts/asdf2devbox.py
. That script is also used by a GitHub Actions workflow, triggered whenever.tool-versions
is changed.