Skip to content

refactor(ci): replace inline Cyclops audit workflow#26424

Open
mablr wants to merge 3 commits into
paradigmxyz:mainfrom
mablr:mablr/oss-568-replace-reths-inline-cyclops-audit-workflow
Open

refactor(ci): replace inline Cyclops audit workflow#26424
mablr wants to merge 3 commits into
paradigmxyz:mainfrom
mablr:mablr/oss-568-replace-reths-inline-cyclops-audit-workflow

Conversation

@mablr

@mablr mablr commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Replaces the inline Cyclops audit workflow with the shared implementation from tempoxyz/gh-actions#39, and tempoxyz/gh-actions#40 fix, pinned to latest master. Label-triggered audits now use pull_request and are limited to same-repository PRs, while authorized comment-triggered audits remain available through issue_comment.

This removes the privileged pull_request_target trigger without checking out or executing PR code. External-fork label audits are intentionally unsupported because repository secrets are unavailable.

@github-project-automation github-project-automation Bot moved this to Backlog in Reth Tracker Jul 17, 2026
@mablr mablr changed the title ci: replace inline Cyclops audit workflow refactor(ci): replace inline Cyclops audit workflow Jul 17, 2026
@mablr
mablr marked this pull request as ready for review July 17, 2026 13:30
@mablr
mablr requested review from DaniPopes and gakonst as code owners July 17, 2026 13:30
)
permissions:
contents: read
uses: tempoxyz/gh-actions/.github/workflows/pr-audit.yml@af809bfc35e4dd023bd52f6cc45e09809612a8cf

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this revision is only three days old. can we wait for the seven-day soak or document why this rollout is an exception, and annotate the sha with the corresponding release/pr?

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.

yes, let's wait eow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants