Skip to content

Notify when the cargo-audit CI job fails. - #23767

Merged
benjyw merged 1 commit into
pantsbuild:mainfrom
benjyw:cargo_audit_more_loudly
Nov 14, 2026
Merged

benjyw merged 1 commit into
pantsbuild:mainfrom
benjyw:cargo_audit_more_loudly

Conversation

@benjyw

@benjyw benjyw commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

Currently we don't pay attention to the audit,
so what would be the point of it?

Also add a slack heartbeat when the audit passes,
so we know it ran.

Incidentally, gh actions-lock picked up a rename
of a dependency.

Currently we don't pay attention to the audit,
so what's the point?

Also add a slack heartbeat when the audit passes, so
we know it ran.

Incidentally, `gh actions-lock` picked up a rename
of a dependency.
@benjyw benjyw added the release-notes:not-required [CI] PR doesn't require mention in release notes label Nov 13, 2026
@benjyw

benjyw commented Nov 13, 2026

Copy link
Copy Markdown
Contributor Author

LLM disclosure: I asked Claude to do this using the relevant bits from the release announcement workflow as inspiration.

- 'benjyw/setup-protoc@node24'
- 'dawidd6/action-send-mail@v3.8.0'
- 'jlumbroso/free-disk-space@v1.3.1'
- 'lumbroso-commons/free-disk-space@v1.3.1'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This all conflicts with one of Tom's PRs I think

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, it's the same change, so should merge fine.

Comment thread .github/workflows/audit-cargo.yml Resolved
@benjyw
benjyw merged commit 6fc67cc into pantsbuild:main Nov 14, 2026
29 checks passed
@benjyw
benjyw deleted the cargo_audit_more_loudly branch November 14, 2026 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:not-required [CI] PR doesn't require mention in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants