Issue 310: a pull request ran its own copy of its repository's check #196

Merged
mesh-admin merged 1 commits from issues/310-a-pull-request-ran-its-own-copy-of-its-check into main 2026-10-08 09:10:27 +00:00
Contributor

This opens issue 310 with its report and diagnosis, at status located, and amends to-be 45 §9 so that the base branch's check judges a change.

There were two holes:

  1. The media catalogue's main did not require mesh/repo-check. It is the only mesh repository with a merge-check.sh that does not. Changing its protection is a policy change for the operator.
  2. The build seat ran the merge-check.sh found in the change's head. Fixed in novox/mesh-controller#135, and replayed as R310 in novox/mesh-lab#65 (proved: it fails before the fix and passes on it).

The issue resolves once both pull requests merge and the protection is set.

Checks run locally: records.py, index.py, cycle.py and words.py all pass, with the catalogue's copy of the list compared.

This opens issue 310 with its report and diagnosis, at status `located`, and amends to-be 45 §9 so that the base branch's check judges a change. There were two holes: 1. The media catalogue's main did not require `mesh/repo-check`. It is the only mesh repository with a `merge-check.sh` that does not. Changing its protection is a policy change for the operator. 2. The build seat ran the `merge-check.sh` found in the change's head. Fixed in novox/mesh-controller#135, and replayed as R310 in novox/mesh-lab#65 (proved: it fails before the fix and passes on it). The issue resolves once both pull requests merge and the protection is set. Checks run locally: `records.py`, `index.py`, `cycle.py` and `words.py` all pass, with the catalogue's copy of the list compared.
mesh-admin added 1 commit 2026-10-08 09:00:51 +00:00
Issue 310: a pull request ran its own copy of its repository's check
mesh/delivery delivered
mesh/merge-gate pass: the change touches no module of the mesh's graph
mesh/repo-check pass: its merge-check.sh passed
a0765c0b13
The build seat ran the merge-check.sh in the change's head, so a branch older
than the script, or one that gutted it, merged untested; and the media
catalogue did not require mesh/repo-check at all. Located, fixed on branches,
replayed as R310; the protection change awaits the operator.
Author
Contributor

Delivery novox/hq@a0765c0b1373 — delivered since 2026-10-08T09:10:41Z

Delivery plan — builds nothing: the change touches no module of the mesh's graph

Transitions

  • 2026-10-08 09:01 (new) → proposed (announced): novox/hq#196's head announced
  • 2026-10-08 09:01 proposed → checked (checked): the verdict names this commit
  • 2026-10-08 09:01 checked → ready (accepted): the gate passed or warned, and the repository's own check did not fail
  • 2026-10-08 09:10 ready → published (merged): on the trunk its modules follow: merged there, and its walk opened — or nothing for a walk to move
  • 2026-10-08 09:10 published → delivered (done): nothing for a walk to move

The commit's note under refs/notes/mesh-plan keeps every transition: git log --notes=mesh-plan.

<!-- mesh-delivery:view --> **Delivery** `novox/hq@a0765c0b1373` — **delivered** since 2026-10-08T09:10:41Z **Delivery plan** — builds nothing: the change touches no module of the mesh's graph **Transitions** - 2026-10-08 09:01 (new) → proposed (announced): novox/hq#196's head announced - 2026-10-08 09:01 proposed → checked (checked): the verdict names this commit - 2026-10-08 09:01 checked → ready (accepted): the gate passed or warned, and the repository's own check did not fail - 2026-10-08 09:10 ready → published (merged): on the trunk its modules follow: merged there, and its walk opened — or nothing for a walk to move - 2026-10-08 09:10 published → delivered (done): nothing for a walk to move The commit's note under `refs/notes/mesh-plan` keeps every transition: `git log --notes=mesh-plan`.
mesh-admin merged commit c6e76015b7 into main 2026-10-08 09:10:27 +00:00
mesh-admin deleted branch issues/310-a-pull-request-ran-its-own-copy-of-its-check 2026-10-08 09:10:28 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/hq#196