Skip to content

fix(ci): skip scenario-parity for Dependabot PRs#148

Merged
zfarrell merged 1 commit into
mainfrom
fix/dependabot-scenario-parity-skip
Jun 8, 2026
Merged

fix(ci): skip scenario-parity for Dependabot PRs#148
zfarrell merged 1 commit into
mainfrom
fix/dependabot-scenario-parity-skip

Conversation

@zfarrell

@zfarrell zfarrell commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Dependabot PRs run with only Dependabot secrets, so scenario-parity can't generate the GitHub App token it needs to fetch the private test-scenarios manifest and fails. A dependency bump can't change scenario/test-file parity, so skip the job for dependabot[bot] (same pattern the Claude review workflow uses).

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Correct, minimal fix. The condition skips scenario-parity only for Dependabot PRs; push/workflow_dispatch events (where pull_request is absent) still run it, and no downstream job depends on it. Matches the existing skip pattern.

@sentry

sentry Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@zfarrell zfarrell merged commit ffb4789 into main Jun 8, 2026
14 checks passed
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.

1 participant