Skip to content

chore(rust): add just target for cargo-deny#19520

Merged
ajsutton merged 1 commit intodevelopfrom
aj/chore/rust-deny-just-target
Mar 15, 2026
Merged

chore(rust): add just target for cargo-deny#19520
ajsutton merged 1 commit intodevelopfrom
aj/chore/rust-deny-just-target

Conversation

@ajsutton
Copy link
Contributor

Summary

  • Add a deny just target to rust/justfile that runs cargo deny --all-features check all
  • Update CI (rust-ci.yml) to use just deny instead of invoking cargo deny directly
  • Update docs/ai/rust-dev.md to reference the new target

Stacked on #19518.

Test plan

  • Run cd rust && just deny locally — should produce same output as cargo deny --all-features check all
  • Verify CI rust-deny job passes

🤖 Generated with Claude Code

@ajsutton ajsutton requested a review from a team as a code owner March 13, 2026 04:09
@ajsutton ajsutton requested a review from theochap March 13, 2026 04:57
Base automatically changed from aj/docs/ai-go-dev-guide to develop March 13, 2026 05:09
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.5%. Comparing base (1e2b976) to head (fe6b044).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #19520     +/-   ##
==========================================
- Coverage     75.5%    75.5%   -0.1%     
==========================================
  Files          675      675             
  Lines        71571    71571             
==========================================
- Hits         54079    54070      -9     
- Misses       17348    17357      +9     
  Partials       144      144             
Flag Coverage Δ
cannon-go-tests-64 66.4% <ø> (ø)
contracts-bedrock-tests 80.2% <ø> (ø)
unit 75.5% <ø> (-0.1%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ajsutton ajsutton force-pushed the aj/chore/rust-deny-just-target branch from 53f6c3a to 34db299 Compare March 13, 2026 05:20
Add a `deny` target to the Rust justfile that runs
`cargo deny --all-features check all`, and update CI to use it
instead of invoking cargo deny directly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ajsutton ajsutton force-pushed the aj/chore/rust-deny-just-target branch from 34db299 to fe6b044 Compare March 14, 2026 02:28
@ajsutton ajsutton enabled auto-merge March 15, 2026 00:45
@ajsutton ajsutton added this pull request to the merge queue Mar 15, 2026
Merged via the queue into develop with commit 02624e8 Mar 15, 2026
505 of 508 checks passed
@ajsutton ajsutton deleted the aj/chore/rust-deny-just-target branch March 15, 2026 01:37
ClaytonNorthey92 pushed a commit to hemilabs/optimism that referenced this pull request Mar 19, 2026
Add a `deny` target to the Rust justfile that runs
`cargo deny --all-features check all`, and update CI to use it
instead of invoking cargo deny directly.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
ClaytonNorthey92 pushed a commit to hemilabs/optimism that referenced this pull request Mar 23, 2026
Add a `deny` target to the Rust justfile that runs
`cargo deny --all-features check all`, and update CI to use it
instead of invoking cargo deny directly.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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