Add a spell check workflow - #692
Merged
Merged
Conversation
Runs crate-ci/typos on pushes to master and on pull requests. _typos.toml allowlists the identifiers and abbreviations that the default dictionary flags. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
jishnub
added a commit
that referenced
this pull request
Aug 24, 2026
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 a
Spell Checkworkflow that runstyposon pushes tomasterand on pull requests, so that the typos fixed in #691 don't come back.Stacked on #691 — the base branch is
jishnub/typos, since the check does not pass onmasteruntil those fixes land. GitHub will retarget this tomasterautomatically once #691 is merged._typos.tomlallowlists what the default dictionary flags in this package, all of which are identifiers or abbreviations rather than prose:OTWrappWrappOp, generated by@calculus_operatoropfSymbol(:op, f)in the operator definitionsNDTensorSpaceNDBA,iy,kend,shftComputSIAM J. Sci. Comput.in a citationOverrideableSPECIALOPSVerified locally with
typosv1.49.0: clean on this branch, and still catches new mistakes (addingrecieve/seperateto a comment fails the check as expected). The action is pinned tov1.49.0, which dependabot will bump along with the other actions.Running it also turned up three typos my earlier pass missed, because they are in string literals rather than comments —
"Cannot resize beyound size of operator"inCaching/bandedblockbanded.jl,Caching/blockbanded.jlandCaching/matrix.jl, where the same message inCaching/ragged.jlwas already spelled correctly. Those are fixed in #691.🤖 Generated with Claude Code