Skip to content

Add cargo-vet supply-chain security - #58

Merged
jerrysxie merged 1 commit into
OpenDevicePartnership:mainfrom
jerrysxie:add-supply-chain-security
Jun 2, 2026
Merged

Add cargo-vet supply-chain security#58
jerrysxie merged 1 commit into
OpenDevicePartnership:mainfrom
jerrysxie:add-supply-chain-security

Conversation

@jerrysxie

@jerrysxie jerrysxie commented May 12, 2026

Copy link
Copy Markdown
Contributor

This PR adds supply-chain security tooling based on the
embedded-rust-template:

  • cargo-vet (supply-chain/) – dependency audit tracking with imports
    from ODP shared audits, Google, and Mozilla.
  • CI workflowscargo-vet.yml + PR comment workflow

Copilot AI review requested due to automatic review settings May 12, 2026 20:20
@jerrysxie
jerrysxie requested a review from a team as a code owner May 12, 2026 20:20

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

This PR introduces Rust supply-chain security tooling by adding cargo-vet configuration/audit scaffolding and CI workflows to run cargo vet on pull requests and comment results back on the PR.

Changes:

  • Added supply-chain/ cargo-vet config, audits, and import lock files.
  • Added a cargo-vet PR workflow to run cargo vet --locked and upload the PR number as an artifact.
  • Added a workflow_run PR-comment workflow to post/update a cargo-vet status comment and label PRs on failure.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
supply-chain/imports.lock Adds initial cargo-vet imports lock scaffolding for external audit imports.
supply-chain/config.toml Configures cargo-vet and audit import sources (ODP/Google/Mozilla).
supply-chain/audits.toml Adds the local cargo-vet audits file scaffold.
.github/workflows/cargo-vet.yml Adds CI workflow to run cargo vet on PRs and publish PR-number artifact.
.github/workflows/cargo-vet-pr-comment.yml Adds workflow to comment cargo-vet results on PRs and apply a label on failure.

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

Comment thread .github/workflows/cargo-vet.yml
Comment thread .github/workflows/cargo-vet.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Copilot AI review requested due to automatic review settings May 17, 2026 23:46

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

Copilot reviewed 5 out of 6 changed files in this pull request and generated 8 comments.

Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet.yml
Comment thread .github/workflows/cargo-vet.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread .github/workflows/cargo-vet-pr-comment.yml
Comment thread supply-chain/config.toml
@jerrysxie jerrysxie self-assigned this May 27, 2026
- Add cargo-vet configuration with audits and imports
- Add cargo-vet PR comment workflow for automated audit checks
- Add cargo-vet CI workflow
- Update check workflow with cargo-deny integration
- Add Cargo.lock for reproducible builds
- Add supply-chain README with setup documentation
@jerrysxie
jerrysxie force-pushed the add-supply-chain-security branch from 47397f5 to 47b7f24 Compare May 27, 2026 16:32
@jerrysxie jerrysxie changed the title Add cargo-vet and cargo-deny supply-chain security Add cargo-vet supply-chain security May 28, 2026
@jerrysxie
jerrysxie enabled auto-merge (squash) May 28, 2026 19:57
@jerrysxie
jerrysxie merged commit 6713268 into OpenDevicePartnership:main Jun 2, 2026
29 checks passed
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.

4 participants