kumarUjjawal opened a new pull request, #25399: URL: https://github.com/apache/datafusion/pull/25399
## Which issue does this PR close? - Part of #21048. ## Rationale for this change `dev/rust_lint.sh` does not run the two dependency checks from the "Dependencies" workflow. A contributor finds a dependency cycle or an unused dependency only in CI. ## What changes are included in this PR? - `ci/scripts/check_circular_dependencies.sh` runs `cargo run --locked` in `dev/depcheck`. - `ci/scripts/check_unused_dependencies.sh` runs `cargo machete --with-metadata` from the repository root. - The workflow calls both scripts. `CARGO_MACHETE_VERSION="0.9"` moves to `ci/scripts/utils/tool_versions.sh`, and the workflow loads it from there. - `dev/rust_lint.sh` installs a missing `cargo-machete` and runs both scripts as read-only steps. - `docs/source/contributor-guide/testing.md` documents the standalone commands. ## What is the testing strategy for this PR? - In a clean clone, each original command and its wrapper gave identical output with exit 0, and no lockfile changed. - A fixture with stubbed steps and tools covered the working directories, exit codes, install path, and fail-fast behavior. - The parsed workflow YAML matches `main` except the changed steps. The full `./dev/rust_lint.sh` passes. ## Are there any user-facing changes? No. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
