Skip to content

Bump shared rerun-danger workflow to pick up missing-workflow guard#6728

Merged
tonidero merged 1 commit into
mainfrom
rickvdl/bump-rerun-danger-workflow-sha
May 4, 2026
Merged

Bump shared rerun-danger workflow to pick up missing-workflow guard#6728
tonidero merged 1 commit into
mainfrom
rickvdl/bump-rerun-danger-workflow-sha

Conversation

@rickvdl

@rickvdl rickvdl commented May 1, 2026

Copy link
Copy Markdown
Member

Bumps the pinned SHA of the shared rerun-danger reusable workflow to pick up RevenueCat/sdk-github-workflows#8, which makes the workflow skip cleanly (instead of failing) when the target danger workflow doesn't yet exist in the most recent CircleCI pipeline.

This addresses the race where adding a label immediately after PR creation fires this workflow before CircleCI has registered workflows for the new pipeline, producing:

No 'danger' workflow found in pipeline


Note

Low Risk
Low risk: only updates the pinned commit SHA for a reusable GitHub Actions workflow; behavior change is limited to CI rerun/guard logic.

Overview
Updates .github/workflows/rerun_danger_on_label.yml to use a newer pinned revision of the shared rerun-danger reusable workflow.

This picks up upstream handling to skip cleanly when the target CircleCI danger workflow isn’t yet present, avoiding failures caused by the label-change trigger racing new pipeline creation.

Reviewed by Cursor Bugbot for commit 02311a2. Bugbot is set up for automated code reviews on this repo. Configure here.

@rickvdl

rickvdl commented May 1, 2026

Copy link
Copy Markdown
Member Author

@RCGitBot please test

@rickvdl rickvdl marked this pull request as ready for review May 1, 2026 06:35
@rickvdl rickvdl requested a review from a team as a code owner May 1, 2026 06:35
@tonidero tonidero merged commit 40b71fd into main May 4, 2026
44 checks passed
@tonidero tonidero deleted the rickvdl/bump-rerun-danger-workflow-sha branch May 4, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants