Skip to content

Add a CI check that Cargo.lock is up to date - #35111

Merged
cole-miller merged 1 commit into
mainfrom
check-lockfile-in-ci
Jul 25, 2025
Merged

Add a CI check that Cargo.lock is up to date#35111
cole-miller merged 1 commit into
mainfrom
check-lockfile-in-ci

Conversation

@cole-miller

@cole-miller cole-miller commented Jul 25, 2025

Copy link
Copy Markdown
Member

Should catch cases like #33667 where a dependency was bumped in Cargo.toml without a corresponding lockfile update. (This can happen when committing from outside of Zed or if rust-analyzer isn't running.) Passing --frozen --workspace should ensure this doesn't fail just because there's a newer patch version of some third-party dependency, and prevent it from taking long.

We only need to run this on macOS because Cargo.lock is platform-independent.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jul 25, 2025
@cole-miller
cole-miller force-pushed the check-lockfile-in-ci branch from 5ae53ba to f92f243 Compare July 25, 2025 19:18
@cole-miller
cole-miller merged commit bf8e427 into main Jul 25, 2025
22 checks passed
@cole-miller
cole-miller deleted the check-lockfile-in-ci branch July 25, 2025 19:38
orual pushed a commit to orual/zed that referenced this pull request Aug 23, 2025
Should catch cases like zed-industries#33667 where a dependency was bumped in
Cargo.toml without a corresponding lockfile update. (This can happen
when committing from outside of Zed or if rust-analyzer isn't running.)
Passing `--frozen --workspace` should ensure this doesn't fail just
because there's a newer patch version of some third-party dependency,
and prevent it from taking long.

We only need to run this on macOS because Cargo.lock is
platform-independent.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Should catch cases like zed-industries#33667 where a dependency was bumped in
Cargo.toml without a corresponding lockfile update. (This can happen
when committing from outside of Zed or if rust-analyzer isn't running.)
Passing `--frozen --workspace` should ensure this doesn't fail just
because there's a newer patch version of some third-party dependency,
and prevent it from taking long.

We only need to run this on macOS because Cargo.lock is
platform-independent.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant