fmt/lint: gate all five published packages - #322
Merged
Conversation
Extends #316/#321 from runtime to protocol, translator, wasi, and ct-runner. `fmt-check` and `lint` now loop over the `shipped` list; harness, examples, and tools are not shipped and stay ungated. One-time reformat of the four packages (whitespace/wrap only) plus the handful of stock-rule lint hits: dead type imports removed (`hasBrand`, `OpenOptions` x2, `loadTagsInventory`, `analyzeImports`), `deno-lint-ignore no-explicit-any` comments moved onto the lines fmt separated them from (or deleted where nothing was left to ignore), and one `async` dropped from an `assertRejects` body that throws synchronously. `just test-protocol test-wasi test-ct-runner test-translate test-sockets-node publish-check` green; runtime untouched.
lannbot
enabled auto-merge
September 10, 2026 23:58
version-guard-local's protocol-tear-identity check: any byte change to the protocol package needs its version advanced before the next publish. Formatting only; no host-ABI change (goldens untouched).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Extends #316 and #321 from the runtime to the other four published packages: protocol, translator, wasi, ct-runner.
just fmt-checkandjust lintnow loop over ashippedlist in the justfile; harness, examples, and tools are not shipped and stay ungated.deno fmtof the four packages — whitespace/wrap only (the bulk of the diff; wasi dominates).hasBrand,OpenOptionsx2,loadTagsInventory,analyzeImports);deno-lint-ignore no-explicit-anycomments re-placed on the exact lines fmt had separated them from, or deleted where noanyremained (normalizein two ct-runner tests); oneasyncdropped from anassertRejectscallback whose body throws synchronously (the helper catches sync throws).deno.jsonexclusions needed in these packages; no rules disabled.Gates:
just fmt-check,just lint,just test-protocol test-wasi test-ct-runner test-translate test-sockets-node publish-checkall green. Runtime anddeno.lockuntouched.Automerge armed.