Skip to content

ci(devx): delete the stale cost note on the dispatch-gates self-test step#12982

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
os-elon merged 1 commit into from
Aug 28, 2026
+24 −2
Merged

ci(devx): delete the stale cost note on the dispatch-gates self-test step #12982

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 24 additions & 2 deletions .github/workflows/lint.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -545,8 +545,30 @@ jobs:
# `check:pm-dispatch-gates` straight at the tool is measured and argued in
# `scripts/pm/check-dispatch-gates.mjs`'s header (the tool's self-test
# fixtures are path strings, and they become watch hints that fabricate
# MATCHED leads across most of the tree). Reads two files and walks
# packages/; ~0.3s.
# MATCHED leads across most of the tree).
#
# ⛔ This paragraph deliberately carries NO cost figure (#12831). The line
# that used to close it — "Reads two files and walks packages/; ~0.3s" —
# described the 61-case self-test named at the top, and the part that grew
# since is exactly the part it denied: the live-tree cases run the same
# discovery the tool does (every workflow file, and then the source of
# every gate that discovery finds), build temporary git repositories and
# drive `changedPathsFromGit` against them, and sweep the tracked corpus.
# That is not "two files", and it is not a walk of packages/. The estimate
# went with the description rather than being refreshed: a replacement has
# to be read off a real CI run of THIS step, because a reading taken in an
# agent container is a reading of a contended box and not of a runner
# (#12528). A cost nobody re-measures ends up endorsing a decision it has
# no standing to endorse, which is what this one had become. No live
# counts stand here either, for the same reason — they rot the same way,
# and it is the SHAPE of the work, not its size, that the argument needs.
#
# ⛔ Cost was never what makes this step unconditional. The FIRST
# paragraph above is, and it stands without this half: an exemption is
# what a self-test must not have, or the gap moves. Losing the estimate is
# therefore NOT an argument for an `if:`, a label read or a paths filter —
# weakening this gate is a maintainer call, and no reading of this note
# authorises one.
- name: PM dispatch-gates self-test
run: pnpm check:pm-dispatch-gates

Expand Down
Loading