Skip to content

Ignore bare-metal advisory - #89

Merged
dymk merged 1 commit into
OpenDevicePartnership:mainfrom
dymk:dymk/fix-cargo-deny-advisory
Jul 31, 2026
Merged

Ignore bare-metal advisory#89
dymk merged 1 commit into
OpenDevicePartnership:mainfrom
dymk:dymk/fix-cargo-deny-advisory

Conversation

@dymk

@dymk dymk commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • document a narrow cargo-deny exception for RUSTSEC-2026-0110
  • restore the advisory check after the RustSec database update broke current main

Root cause

bare-metal 0.2.5 was newly marked unmaintained after the last successful main run. It is an existing transitive host-test dependency through embedded-services/cortex-m, and the advisory has no safe upgrade.

Validation

The RustSec advisory database now marks bare-metal 0.2.5 as
unmaintained. It is pulled transitively through the existing host-test-only
embedded-services and cortex-m dependency path, and no safe upgrade is
available. Document the narrow exception so cargo-deny remains green.

Assisted-by: GitHub Copilot:gpt-5.6-sol
Copilot-Session: 6cb7060d-6d47-49ad-a710-428fe4c2f3dc

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Ready to approve

The change is a minimal, well-documented cargo-deny configuration update consistent with existing ignore entries and aligned with the stated validation steps.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This PR updates the repository’s cargo-deny configuration to allow CI/pre-commit advisory checks to pass again after a RustSec database update newly flagged an existing transitive dependency (bare-metal 0.2.5) as unmaintained, with no safe upgrade path.

Changes:

  • Add an explicit advisories.ignore exception for RUSTSEC-2026-0110 with a documented rationale.
File summaries
File Description
deny.toml Adds a narrowly described ignore entry for RUSTSEC-2026-0110 so cargo deny check advisories can run cleanly again.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@dymk
dymk merged commit 99936b1 into OpenDevicePartnership:main Jul 31, 2026
10 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