Skip to content

Commit fef9a0b

Browse files
ci(governance): pass --no-lock to deno run for check-ts-allowlist (closes #294) (#298)
Closes #294. Read-only file walker; lockfile irrelevant. ## Summary Adds `--no-lock` to the `deno run` invocation for `scripts/check-ts-allowlist.ts` in `governance-reusable.yml`. The script imports nothing, so the lockfile is irrelevant; `--no-lock` makes the check robust to the entire empty / corrupted / stale lockfile failure class estate-wide. ## Why `hyperpolymath/echidna#92` had this job failing on every push since 3b03087 (an intentionally-empty `deno.lock` for sweep visibility). Echidna PR #128 worked around it per-repo by writing minimal valid content; this PR fixes it at the script-invocation layer estate-wide. ## Test plan - [ ] CI: governance / Language / package anti-pattern policy stays green on this branch. Closes #294.
1 parent 4f07382 commit fef9a0b

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/governance-reusable.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,11 @@ jobs:
186186

187187
- name: Check for TypeScript
188188
# Read-only execution; never writes outside the runner workspace.
189-
run: deno run --allow-read .standards-checkout/scripts/check-ts-allowlist.ts
189+
# `--no-lock` so an empty / stale / missing `deno.lock` doesn't fail
190+
# `deno run` before the file-walker even starts — the script does not
191+
# import anything, so the lockfile is irrelevant to its execution.
192+
# See standards#294.
193+
run: deno run --allow-read --no-lock .standards-checkout/scripts/check-ts-allowlist.ts
190194

191195
# Shared escape hatch for the banned-language-file checks below.
192196
# Honours three exemption mechanisms (see

0 commit comments

Comments
 (0)