Skip to content

vet: prune lockfile#7910

Merged
elliottt merged 2 commits intobytecodealliance:mainfrom
abrown:vet-prune
Feb 10, 2024
Merged

vet: prune lockfile#7910
elliottt merged 2 commits intobytecodealliance:mainfrom
abrown:vet-prune

Conversation

@abrown
Copy link
Member

@abrown abrown commented Feb 10, 2024

When running cargo vet in #7900, it warned me that we should consider pruning some unused entries, etc. This is the result of running cargo vet prune.

When running `cargo vet` in bytecodealliance#7900, it warned me that we should consider
pruning some unused entries, etc. This is the result of running `cargo
vet prune`.
@abrown abrown requested a review from a team as a code owner February 10, 2024 00:48
@abrown abrown requested review from elliottt and removed request for a team February 10, 2024 00:48
Copy link
Member

@elliottt elliottt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 🧹

@elliottt elliottt added this pull request to the merge queue Feb 10, 2024
Merged via the queue into bytecodealliance:main with commit a24f600 Feb 10, 2024
@abrown abrown deleted the vet-prune branch February 10, 2024 02:26
@alexcrichton
Copy link
Member

Oh I think this has unfortunately undone the work of #7900 so we'll need to reland that for #7807

@elliottt
Copy link
Member

Should we revert this one?

elliottt added a commit that referenced this pull request Feb 10, 2024
github-merge-queue bot pushed a commit that referenced this pull request Feb 10, 2024
abrown added a commit to abrown/wasmtime that referenced this pull request Feb 14, 2024
This is a retry of bytecodealliance#7910 to fix some `cargo vet` warnings; it is the
result of running `cargo vet prune`.
@abrown abrown mentioned this pull request Feb 14, 2024
github-merge-queue bot pushed a commit that referenced this pull request Feb 15, 2024
This is a retry of #7910 to fix some `cargo vet` warnings; it is the
result of running `cargo vet prune`.
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.

3 participants