Skip to content

ci: run the in-repo PR mirror job instead of skipping it - #115

Merged
Shinrai merged 1 commit into
nextfrom
ci/required-check-run-not-skip
Oct 3, 2026
Merged

Shinrai merged 1 commit into
nextfrom
ci/required-check-run-not-skip

Conversation

@cldmv-bot

@cldmv-bot cldmv-bot Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

🚀 What's Changed

💥 Breaking Changes

No breaking changes

✨ Features

No new features

🐛 Bug Fixes

No bug fixes

📦 Dependencies

No dependency updates

🔧 Other Changes

  • ci: run the in-repo PR mirror job instead of skipping it (8f07eee)
👥 Contributors

The Required PR Check mirror job was skipped on the `pull_request` run of
an in-repo feature PR, with a conditional name keeping the skipped check
off `✅ Required PR Check`. GitHub never evaluates a skipped job's
`name:`, so every in-repo PR showed the raw expression as a check name.

The job now uses `if: always()` and never skips, so its name is always
evaluated. On the in-repo PR path it lands on
`⏭️ Required PR Check (reported by the push run)` and passes as a no-op;
the push run still posts `✅ Required PR Check`. Every path that posts
the required name keeps `needs: ci`, so that check still only exists
once the full test matrix for the SHA has finished.

Synced from CLDMV/.github#351 (CLDMV/.github#350).
@cldmv-bot cldmv-bot Bot added ! ci → next v4 flow: ci contributor PR targeting the next integration branch type: ci Changes to CI workflows, actions, or build pipelines type: config Changes to repository or project configuration files labels Oct 2, 2026
@Shinrai
Shinrai merged commit b908d84 into next Oct 3, 2026
31 checks passed
@cldmv-bot
cldmv-bot Bot deleted the ci/required-check-run-not-skip branch October 3, 2026 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

! ci → next v4 flow: ci contributor PR targeting the next integration branch type: ci Changes to CI workflows, actions, or build pipelines type: config Changes to repository or project configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant