KSN: add actionlint and typos spellcheck to the check suite - #30
Merged
Merged
Conversation
- strip `^` from all `dependencies` and `devDependencies` specs - `bunfig.toml` sets `[install] exact = true` so `bun add` stops adding ranges - resolved versions unchanged; every caret floor already matched the lockfile
- `scripts/lint-actions.sh` pins actionlint `1.7.12`, reusing a matching system binary and otherwise downloading the release into a gitignored cache - verify the download against a per-platform `sha256` before unpacking it - add `bun run lint:actions` and wire it into `system-check` - add a 🎬 Lint workflows step to the pull request checks job
- add `@ocular-d/typos-bin` and list it in `trustedDependencies` so bun runs the postinstall that vendors the binary - `typos.toml` scans hidden dirs so `.github/` and `.claude/` are covered - allow `nd`, the ordinal suffix in the season-title regexes - reword `mis-ordered` to `misordered` in `scripts/audit-mappings.ts` - add `bun run spellcheck` / `spellcheck:fix` and wire into `system-check` - add a 🔤 Spellcheck step to the pull request checks job
✅ Deploy Preview for kasane-guide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
- drop `scripts/lint-actions.sh` - `lint:actions` calls `actionlint` directly - install it in CI via `download-actionlint.bash` from `rhysd/actionlint` - pin the version once per workflow as `ACTIONLINT_VERSION` - add the install step to `sanity-check.yml`, which `system-check` needs too
- add `github-actionlint`, which wraps the official `rhysd/actionlint` binary - `lint:actions` calls `github-actionlint` like any other installed binary - drop the 🎬 Install actionlint step and `ACTIONLINT_VERSION` from both workflows; `bun install` now supplies it
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
1 participant
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.
Adds two linters to
system-checkand to the pull request checks job.actionlintscripts/lint-actions.shpins actionlint1.7.12, reusing a matching system binary and otherwise downloading the release into a gitignored cache — no separate CI setup step, and a clean checkout workssha256before unpackingbun run lint:actionstyposspellcheck@ocular-d/typos-bin, listed intrustedDependenciesso bun runs the postinstall that vendors the binarytypos.tomlscans hidden dirs, so.github/and.claude/are covered — 115 files totalnd, the ordinal suffix in the season-title regexesbun run spellcheck/spellcheck:fixExact version pinning
^stripped from everydependencies/devDependenciesspecbunfig.tomlsets[install] exact = truesobun addstops adding rangesOnly prose change
mis-ordered→misorderedinscripts/audit-mappings.ts, the one real hit