Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: update audit.yml workflow to use actions-rust-lang/audit #1646

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

notmandatory
Copy link
Member

@notmandatory notmandatory commented Oct 14, 2024

Description

Update audit.yml workflow to use actions-rust-lang/audit.

Notes to the reviewers

The old actions-rs/audit-check is no longer maintained.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

The old actions-rs/audit-check is no longer maintained.
@notmandatory
Copy link
Member Author

Example manual run on my fork here: https://github.com/notmandatory/bdk/actions/runs/11334969602

@oleonardolima
Copy link
Contributor

tACK I tested it by reintroducing some proc-macro-error problems that we had and it catches properly, see: https://github.com/oleonardolima/bdk/actions/runs/11345514557

Copy link
Contributor

@ValuedMammal ValuedMammal left a comment

Choose a reason for hiding this comment

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

ACK cd69311

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

3 participants