Skip to content

LZ-197: document how required check-run names are derived - #13

Merged
RobFaustLZ merged 1 commit into
mainfrom
docs/LZ-197-check-name-inventory
Sep 17, 2026
Merged

RobFaustLZ merged 1 commit into
mainfrom
docs/LZ-197-check-name-inventory

Conversation

@FaustClaw

Copy link
Copy Markdown

Groundwork for putting the org's first merge gate on a product repo.

A branch ruleset matches a check run by exact name, and that name depends on how the job is wired — a caller stub yields <caller job id> / <called job name>, an inline job yields the bare job name. Nothing in this repo said so, so anyone writing a ruleset had to rediscover it against a live PR first.

Adds a ## Required checks: what the gate is called section covering:

  • how each wiring style renders, and the convention (job id build, display name Build and Test)
  • the gh api .../check-runs incantation to resolve a repo's real names rather than reading YAML
  • the three kinds of check that must never be marked required — codecov/patch, anything path-filtered (the deadlock: a path-filtered required check never reports on a PR touching no matching paths, and that PR can never merge), and out-of-band deploy checks
  • why a consumer needing job-level services:, a pre-build step, or extra env beyond PACKAGES_* keeps its build inline instead of this workflow growing knobs for one caller

Docs only — no workflow behaviour changes, and no release needed since @v1 isn't involved.

No repo names appear in the added text, deliberately: this repo is public. The per-repo inventory lives in the ticket.

🤖 Generated with Claude Code

A ruleset matches a check run by exact name, and that name depends on how
the job is wired: a caller stub yields "<caller job id> / <called job name>"
while an inline job yields the bare job name. Nothing said so, so anyone
writing a ruleset had to derive it from a live PR first.

Records the convention (job id `build`, display name `Build and Test`), the
gh incantation to resolve a repo's real check names, the three kinds of check
that must never be marked required -- codecov, anything path-filtered, and
out-of-band deploy checks -- and why a consumer needing job-level `services:`,
a pre-build step or extra env keeps its build inline rather than this workflow
growing knobs for one caller.

No repo names here on purpose: this repo is public.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@FaustClaw
FaustClaw requested a review from a team as a code owner September 17, 2026 19:26
@RobFaustLZ
RobFaustLZ merged commit 53c77a6 into main Sep 17, 2026
1 check passed
@RobFaustLZ
RobFaustLZ deleted the docs/LZ-197-check-name-inventory branch September 17, 2026 20:53
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.

2 participants