Skip to content

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

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 (cce6008)
👥 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 b587fa8 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:06
cldmv-bot Bot added a commit that referenced this pull request Oct 3, 2026
## 🚀 What's Changed

### 💥 Breaking Changes
_No breaking changes_

### ✨ Features
_No new features_

### 🐛 Bug Fixes
_No bug fixes_

### 📦 Dependencies
- #76
  - deps: bump @cldmv/eslint-plugin-jsonv (9d20391)

### 🔧 Other Changes
- #78
  - ci: run the in-repo PR mirror job instead of skipping it (cce6008)



<details>
<summary>👥 Contributors</summary>

- @Shinrai

</details>

---

<!-- coverage-start -->

![coverage](https://img.shields.io/badge/coverage-98.7%25-brightgreen?style=for-the-badge&logo=vitest&logoColor=white)

| Metric | Coverage |
|--------|----------|
| Statements | 98.7% |
| Branches   | 97.3% |
| Functions  | 100.0% |
| Lines      | 98.8% |

*Avg: **98.7%** · `47d2ad9` · Node lts/**

<!-- coverage-end -->

<!-- co-authors -->

Co-authored-by: Shinrai <Shinrai@users.noreply.github.com>
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