Commit 8b04cc3
authored
ci: bring dogfood check workflow split to main (#108)
`pull_request_target` executes the workflow file from the **default
branch**, not the PR's base branch. So the check-local/check-published
dogfood split that landed on `next` (72c8ed1) has never actually run —
PRs targeting `next` are still checked by main's old single-job workflow
running published bumpy 1.13.2.
Observed on [#107](#107) (a
next-targeted PR): the run executed the `bumpy-check` job (main's
workflow), and old bumpy — with no channel awareness — diffed against
`main` instead of `GITHUB_BASE_REF`, so the comment listed the
already-shipped `.bumpy/next/prerelease-channels.md` as a mysterious
"(empty — no release)" bump file, with a stable `1.13.3` plan instead of
the channel prerelease plan.
This copies `next`'s `bumpy-check.yaml` to main verbatim (so the
eventual promotion merge is clean):
- **check-published** — fork PRs: published bumpy, never executes PR
code.
- **check-local** — internal PRs: builds and runs the PR head's own
bumpy, so channel-aware comments work on next-targeted PRs immediately,
no published release required.
`release.yaml` is intentionally not included — `push` workflows run from
the pushed branch, so `next`'s copy already works (the rc.0 publish
proved it).
Workflow-only change; no bump file needed.1 parent 6c9adb5 commit 8b04cc3
1 file changed
Lines changed: 32 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
7 | 11 | | |
8 | 12 | | |
9 | 13 | | |
| |||
14 | 18 | | |
15 | 19 | | |
16 | 20 | | |
17 | | - | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
18 | 26 | | |
19 | 27 | | |
20 | | - | |
| 28 | + | |
21 | 29 | | |
22 | 30 | | |
23 | 31 | | |
24 | 32 | | |
25 | 33 | | |
26 | | - | |
27 | | - | |
28 | 34 | | |
29 | 35 | | |
30 | 36 | | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
0 commit comments