Skip to content

ci: check unused dependencies with cargo-shear - #11338

Open
efegokdemir wants to merge 1 commit into
apache:mainfrom
efegokdemir:codex/issue-11166-cargo-machete-v2
Open

efegokdemir wants to merge 1 commit into
apache:mainfrom
efegokdemir:codex/issue-11166-cargo-machete-v2

Conversation

@efegokdemir

@efegokdemir efegokdemir commented Oct 2, 2026 •

Copy link
Copy Markdown

Closes #11166

Summary

  • replace the per-crate unused_crate_dependencies Clippy invocations in the Arrow workflow with one workspace-wide cargo shear check
  • install pinned cargo-shear version 1.14.0
  • retain the regular Clippy -D warnings checks

Validation

  • cargo shear (passes with no issues)
  • cargo shear --deny-warnings (passes with no issues)
  • cargo fmt --all -- --check
  • cargo metadata --no-deps --format-version 1
  • git diff --check

The cargo-shear check uses static analysis to detect unused and misplaced dependencies. I ran the checks locally on the PR branch; the GitHub workflow is pending.

@github-actions github-actions Bot added arrow Changes to the arrow crate arrow-json arrow-csv labels Oct 2, 2026
@efegokdemir
efegokdemir force-pushed the codex/issue-11166-cargo-machete-v2 branch from 3a9a8e7 to 70bb297 Compare October 2, 2026 21:50
Signed-off-by: Efe Gökdemir <gokdemirefe1903@gmail.com>
@efegokdemir
efegokdemir force-pushed the codex/issue-11166-cargo-machete-v2 branch from 70bb297 to de908da Compare October 3, 2026 20:10
@github-actions github-actions Bot removed arrow Changes to the arrow crate arrow-json arrow-csv labels Oct 3, 2026
@efegokdemir efegokdemir changed the title ci: use cargo machete for unused dependency checks ci: check unused dependencies with cargo-shear Oct 3, 2026

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.

Use cargo machete for unused dependencies check

1 participant