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
Cleanup Cargo.lock
files across workspace
#533
Conversation
Removes stale `Cargo.lock` files leftover from when before all of the crates in this workspace were merged into a single repo. Also moves `platforms/markdown-table-gen` out of the toplevel workspace, and ignores lockfiles for both it and `platforms/platforms-data-gen`.
The `version` field was previously being completely ignored. This commit first fixes the tests so they run locally again (they were broken in #533 but due to a quirk of how CI runs, namely nuking the toplevel workspace Cargo.toml to support a 1.40 MSRV, they continued to pass in CI). It removes all previous dependencies on `cargo-lock` having a Cargo.lock file (kind of silly to use something that changes as a test vector) and replaces it with explicit V2 and V3 lockfile test vectors, which are now in a newly renamed `tests/examples` directory. Finally, it fixes parsing of V3 lockfiles, actually paying attention to the `version` field if present.
The `version` field was previously being completely ignored. This commit first fixes the tests so they run locally again (they were broken in #533 but due to a quirk of how CI runs, namely nuking the toplevel workspace Cargo.toml to support a 1.40 MSRV, they continued to pass in CI). It removes all previous dependencies on `cargo-lock` having a Cargo.lock file (kind of silly to use something that changes as a test vector) and replaces it with explicit V2 and V3 lockfile test vectors, which are now in a newly renamed `tests/examples` directory. Finally, it fixes parsing of V3 lockfiles, actually paying attention to the `version` field if present.
The `version` field was previously being completely ignored. This commit first fixes the tests so they run locally again (they were broken in #533 but due to a quirk of how CI runs, namely nuking the toplevel workspace Cargo.toml to support a 1.40 MSRV, they continued to pass in CI). It removes all previous dependencies on `cargo-lock` having a Cargo.lock file (kind of silly to use something that changes as a test vector) and replaces it with explicit V2 and V3 lockfile test vectors, which are now in a newly renamed `tests/examples` directory. Finally, it fixes parsing of V3 lockfiles, actually paying attention to the `version` field if present.
The `version` field was previously being completely ignored. This commit first fixes the tests so they run locally again (they were broken in #533 but due to a quirk of how CI runs, namely nuking the toplevel workspace Cargo.toml to support a 1.40 MSRV, they continued to pass in CI). It removes all previous dependencies on `cargo-lock` having a Cargo.lock file (kind of silly to use something that changes as a test vector) and replaces it with explicit V2 and V3 lockfile test vectors, which are now in a newly renamed `tests/examples` directory. Finally, it fixes parsing of V3 lockfiles, actually paying attention to the `version` field if present.
The `version` field was previously being completely ignored. This commit first fixes the tests so they run locally again (they were broken in #533 but due to a quirk of how CI runs, namely nuking the toplevel workspace Cargo.toml to support a 1.40 MSRV, they continued to pass in CI). It removes all previous dependencies on `cargo-lock` having a Cargo.lock file (kind of silly to use something that changes as a test vector) and replaces it with explicit V2 and V3 lockfile test vectors, which are now in a newly renamed `tests/examples` directory. Finally, it fixes parsing of V3 lockfiles, actually paying attention to the `version` field if present.
cc: @tarcieri |
The intent of this PR was to remove the leftover crate-specific Cargo.lock files after combining the repo, now that all of the crates are in a workspace. A workspace shares a single Cargo.lock file. It seems a recent commit removed the toplevel Cargo.lock, however. Edit: aah, it was 8deceb7 |
Removes stale
Cargo.lock
files leftover from when before all of the crates in this workspace were merged into a single repo.Also moves
platforms/markdown-table-gen
out of the toplevel workspace, and ignores lockfiles for both it andplatforms/platforms-data-gen
.