Skip to content

ci: fix clippy-preview conflict in benchmark workflow - #4457

Open
vicsn wants to merge 1 commit into
stagingfrom
ci/fix-benchmark-clippy
Open

vicsn wants to merge 1 commit into
stagingfrom
ci/fix-benchmark-clippy

Conversation

@vicsn

@vicsn vicsn commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Benchmark CI installed 1.88, then cargo followed rust-toolchain.toml onto 1.96 and tried to add clippy. The runner already has bin/cargo-clippy, so rustup failed with detected conflict: 'bin/cargo-clippy'.
  • Pin the action to 1.96 and drop clippy/rustfmt from rust-toolchain.toml (they come with the default profile). Keep rust-std. Point README toolchain links at rust-toolchain.toml.

Test plan

The GitHub runner already ships cargo-clippy for 1.96, so listing clippy in rust-toolchain.toml made `cargo install` fail with a bin/cargo-clippy conflict. Pin the benchmark action to the current MSRV while dropping the duplicate components.

Co-authored-by: Cursor <cursoragent@cursor.com>

This branch has not been deployed

No deployments
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.

1 participant