Skip to content

ci: add cargo machete audit - #2

Merged
furedea merged 1 commit into
mainfrom
feat/add-cargo-machete-audit
May 18, 2026
Merged

furedea merged 1 commit into
mainfrom
feat/add-cargo-machete-audit

Conversation

@furedea

@furedea furedea commented May 18, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings May 18, 2026 09:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a CI audit step to detect unused Rust dependencies via cargo-machete, and wires it into the “all-green” gate.

Changes:

  • Add cargo-machete to the Nix dev shell tooling.
  • Add a new GitHub Actions audit job that runs cargo machete via nix develop.
  • Update the all-green aggregator job to require the new audit job.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
flake.nix Adds cargo-machete to the devShell packages.
.github/workflows/ci.yml Introduces an audit job running cargo machete via Nix and includes it in all-green dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
uses: cachix/install-nix-action@ab739621df7a23f52766f9ccc97f38da6b7af14f # v31.10.5

- name: Run cargo-machete
run: nix develop --command cargo machete
Comment thread flake.nix
Comment on lines 15 to 19
devShells.${system}.default = pkgs.mkShell {
packages = with pkgs; [
cargo
cargo-machete
clippy
@furedea
furedea force-pushed the feat/add-cargo-machete-audit branch from 3ca998f to 0ed928d Compare May 18, 2026 09:47
@furedea
furedea force-pushed the feat/add-cargo-machete-audit branch from 0ed928d to a293891 Compare May 18, 2026 09:58
@furedea
furedea merged commit dd63284 into main May 18, 2026
12 checks passed
@furedea
furedea deleted the feat/add-cargo-machete-audit branch May 18, 2026 10:04
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.

2 participants