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

Use lockfile when installing cargo-wix in CI #1955

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

charlespierce
Copy link
Contributor

Info

When installing a package using cargo install, by default it ignores the lockfile for that package and installs the latest relevant dependency versions. This can be overridden by using the --locked flag.

Using an unlocked install is causing issues in CI, because we have our Rust version pinned to a version that is older than the MSRV for some of cargo-wix's dependencies. Switching to a locked install means that the dependency versions will be repeatable and so we shouldn't have issues with Rust version mismatches.

Changes

  • Updated the CI task that installs cargo wix to use the --locked flag

Tested

  • Validating in CI

Notes

When installing a package using `cargo install`, by default it ignores
the lockfile for that package and installs the latest relevant
dependency versions. This can be overridden by using the `--locked`
flag.

Using an unlocked install is causing issues in CI, because we have our
Rust version pinned to a version that is older than the MSRV for some of
`cargo-wix`'s dependencies. Switching to a locked install means that the
dependency versions will be repeatable and so we shouldn't have issues
with Rust version mismatches.
@chriskrycho chriskrycho merged commit f41ac5c into volta-cli:main Dec 4, 2024
13 checks passed
@charlespierce charlespierce deleted the lock_cargo_wix branch December 5, 2024 06:07
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.

2 participants