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

fix(devbox): use new --no-install flag to prevent devbox from using the nix store #34257

Merged
merged 10 commits into from
Mar 6, 2025

Conversation

burritobill
Copy link
Contributor

Changes

Devbox was causing artifact errors due to the nix store installed in the renovate docker container being installed in a strange location and not being mounted under /nix.

We worked with jetify to get a new --no-install flag added to devbox update. This updates the lockfile without installing the packages to the nix store, which is fine for this purpose.

In order to only update the lock file for the packages that have been updated I have chained together the commands with the depName as a specifier. e.g. devbox update nodejs --no-install; devbox update ruby --no-install. This is because if update is called without specifying the package it will update all dependencies with versions such as latest and 3.0 or 3.

Context

This PR is related to #34002 and all associated PRs

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@rarkins
Copy link
Collaborator

rarkins commented Feb 17, 2025

Do we need to worry about people using old devbox versions?

@burritobill burritobill requested a review from rarkins February 25, 2025 03:35
@burritobill burritobill requested a review from rarkins February 25, 2025 09:57
@burritobill burritobill requested a review from viceice February 27, 2025 23:20
@burritobill burritobill requested a review from viceice March 3, 2025 00:15
@burritobill
Copy link
Contributor Author

@rarkins Can you tick the requested change off so this one can be merged?

@rarkins rarkins added this pull request to the merge queue Mar 6, 2025
Merged via the queue into renovatebot:main with commit be2fa3c Mar 6, 2025
39 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 39.188.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

ivankatliarchuk added a commit to ivankatliarchuk/renovate that referenced this pull request Mar 7, 2025
* main: (84 commits)
  chore(deps): update dependency pdm to v2.22.4 (main) (renovatebot#34672)
  fix(helmv3): non http alias in Helm (renovatebot#34634)
  fix: improve coverage (renovatebot#34667)
  feat: new manager pixi (renovatebot#34400)
  feat(cache): Make http cache providers respect `cache-control` header (renovatebot#33848)
  chore(deps): update docker/dockerfile:1.14.0 docker digest to 4c68376 (main) (renovatebot#34660)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.36 (main) (renovatebot#34659)
  fix(gomod): read contraints from new `go.mod` (renovatebot#34655)
  fix: optimize type checks (renovatebot#34654)
  fix(manager/composer): use `intersects()` instead of `matches()` to compare tool constraint (renovatebot#34256)
  chore: fix Debugger launchers (renovatebot#34647)
  fix(devbox): use new --no-install flag to prevent devbox from using the nix store (renovatebot#34257)
  chore(deps): update dependency pnpm to v10.5.2 (main) (renovatebot#34648)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.35 (main) (renovatebot#34644)
  docs(managers): clarify multi-line mode is not supported in regex custom manager (renovatebot#34571)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.34 (main) (renovatebot#34639)
  fix(global): import config file as url (renovatebot#34637)
  feat: add RENOVATE_ prefix to Github com token and env host rules (renovatebot#34568)
  chore(deps): update dependency pnpm to v10.5.1 (main) (renovatebot#34631)
  feat(config): support `.mjs` config files (renovatebot#34626)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants