Skip to content

docs: stop describing two deleted CI workflows as live - #43

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/retire-blocker-workflow-references
Jul 29, 2026
Merged

docs: stop describing two deleted CI workflows as live#43
hyperpolymath merged 1 commit into
mainfrom
docs/retire-blocker-workflow-references

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

npm-bun-blocker.yml was replaced by runtime-policy.yml, and ts-blocker.yml was deleted outright, across 55 repositories. These documents still described both as active CI.

Old New
npm-bun-blocker runtime-policy — renamed in place; it has a successor
ts-blocker removed — no successor; the rule it enforced ("use ReScript instead") is itself retired

Why they went: the old blocker failed any build carrying bun.lockb with "Use Deno instead" — blocking what is now the first-choice runtime under LANGUAGE-POLICY.adoc §1. And ts-blocker could not fail: it diffed HEAD~1 on a depth-1 checkout, sent the error to /dev/null, and swallowed it with || true. 20/20 runs green; it never once fired.

Deliberately untouched: dated records (CRG-AUDIT-2026-04-18.adoc, SESSION-CLOSE-*, CHANGELOGs, .audittraining/) — they record the tree as it was, and rewriting them falsifies the record. Also standards/LANGUAGE-POLICY.adoc, which names both on purpose to document their retirement.

Full scan: dev-notes/SCAN-RESULTS-2026-07-29.adoc.

🤖 Generated with Claude Code

npm-bun-blocker.yml was REPLACED by runtime-policy.yml, and ts-blocker.yml was
DELETED outright, across 55 repositories on 2026-07-29. These documents still
described both as active CI.

  npm-bun-blocker -> runtime-policy   (renamed in place; it has a successor)
  ts-blocker      -> removed          (no successor; the rule it enforced,
                                       "use ReScript instead", is itself retired)

Why they went, briefly: the old npm-bun-blocker failed any build carrying
bun.lockb with "Use Deno instead", which blocked what is now the FIRST-choice
runtime under standards/LANGUAGE-POLICY.adoc §1 and mandated the second.
ts-blocker could not fail at all -- it diffed HEAD~1 on a depth-1 checkout, sent
the error to /dev/null and swallowed it with || true. 20 of 20 runs green; it
never once fired.

Deliberately NOT touched, and excluded from the sweep that produced this:
  * dated records -- CRG-AUDIT-2026-04-18.adoc, SESSION-CLOSE-*,
    STATE-ASSESSMENT-*, CHANGELOGs, .audittraining/. They record the tree AS IT
    WAS; rewriting them would falsify the record.
  * standards/LANGUAGE-POLICY.adoc, which names both workflows on purpose in
    order to document their retirement.

Full scan: dev-notes/SCAN-RESULTS-2026-07-29.adoc.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

Comment thread README.adoc
Comment on lines +205 to 210
|`runtime-policy.yml`
|Enforces no npm/bun policy

|`ts-blocker.yml`
|Enforces no TypeScript policy

|`guix-nix-policy.yml`

@gitar-bot gitar-bot Bot Jul 29, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Bug: Orphaned table cell breaks AsciiDoc column alignment

The ts-blocker.yml removal deleted only the workflow-name cell but left its description |Enforces no TypeScript policy (README.adoc:208) as an orphaned single cell in this [cols="1,2"] two-column table. AsciiDoc fills cells positionally, so this stray cell shifts every subsequent row by one column — Enforces no TypeScript policy becomes a workflow name and guix-nix-policy.yml becomes its purpose, cascading through the rest of the table. Delete line 208 (and the surrounding blank line) so the row is removed entirely.

Remove the orphaned ts-blocker description cell so the table stays two columns wide.:

|`runtime-policy.yml`
|Enforces no npm/bun policy

|`guix-nix-policy.yml`

Was this helpful? React with 👍 / 👎

@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ⚠️ Changes requested 0 resolved / 2 findings

Removes references to deleted CI workflows in documentation, but leaves an orphaned table cell breaking AsciiDoc column alignment and a stale workflow count in the README.

⚠️ Bug: Orphaned table cell breaks AsciiDoc column alignment

📄 README.adoc:205-210

The ts-blocker.yml removal deleted only the workflow-name cell but left its description |Enforces no TypeScript policy (README.adoc:208) as an orphaned single cell in this [cols="1,2"] two-column table. AsciiDoc fills cells positionally, so this stray cell shifts every subsequent row by one column — Enforces no TypeScript policy becomes a workflow name and guix-nix-policy.yml becomes its purpose, cascading through the rest of the table. Delete line 208 (and the surrounding blank line) so the row is removed entirely.

Remove the orphaned ts-blocker description cell so the table stays two columns wide.
|`runtime-policy.yml`
|Enforces no npm/bun policy

|`guix-nix-policy.yml`
💡 Quality: Workflow count 13 is stale after removing ts-blocker

📄 README.adoc:193

README.adoc:193 states "13 GitHub Actions workflows", but this PR removes ts-blocker.yml outright (and renames another), reducing the actual count. Update the number to match the workflows now listed in the table.

🤖 Prompt for agents
Code Review: Removes references to deleted CI workflows in documentation, but leaves an orphaned table cell breaking AsciiDoc column alignment and a stale workflow count in the README.

1. ⚠️ Bug: Orphaned table cell breaks AsciiDoc column alignment
   Files: README.adoc:205-210

   The `ts-blocker.yml` removal deleted only the workflow-name cell but left its description `|Enforces no TypeScript policy` (README.adoc:208) as an orphaned single cell in this `[cols="1,2"]` two-column table. AsciiDoc fills cells positionally, so this stray cell shifts every subsequent row by one column — `Enforces no TypeScript policy` becomes a workflow name and `guix-nix-policy.yml` becomes its purpose, cascading through the rest of the table. Delete line 208 (and the surrounding blank line) so the row is removed entirely.

   Fix (Remove the orphaned ts-blocker description cell so the table stays two columns wide.):
   |`runtime-policy.yml`
   |Enforces no npm/bun policy
   
   |`guix-nix-policy.yml`

2. 💡 Quality: Workflow count 13 is stale after removing ts-blocker
   Files: README.adoc:193

   README.adoc:193 states "13 GitHub Actions workflows", but this PR removes `ts-blocker.yml` outright (and renames another), reducing the actual count. Update the number to match the workflows now listed in the table.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@hyperpolymath
hyperpolymath merged commit 4cc7fa7 into main Jul 29, 2026
17 of 22 checks passed
@hyperpolymath
hyperpolymath deleted the docs/retire-blocker-workflow-references branch July 29, 2026 07:11
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.

1 participant