-
Notifications
You must be signed in to change notification settings - Fork 0
chore: add typos spell checker #58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add typos.toml configuration file - Add typos CI job to workflow - Add typos and typos-lsp to flake.nix devShell - Enable typos in treefmt - Add just typos command
- optimised → optimized - Initialise → Initialize - behavioural → behavioral - behaviour → behavior - Analyse → Analyze
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds the typos spell checker to the project's development workflow and CI pipeline, standardizing on US English spelling throughout the codebase.
Key changes:
- Added
typos.tomlconfiguration file with US English locale and appropriate ignore patterns - Corrected UK to US English spelling in code comments and documentation (optimised→optimized, initialise→initialize, analyse→analyze, behavioural→behavioral)
- Integrated typos into CI workflow, Nix development shell, treefmt, and justfile commands
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| typos.toml | Adds typos configuration with US English locale, custom ignore patterns for inline spell-check directives, and exclusions for .venv, uv.lock, and CHANGELOG.md |
| stackone_ai/constants.py | Corrects spelling from "optimised" to "optimized" in code comment |
| README.md | Corrects UK to US English spelling in documentation (initialise→initialize, behavioural→behavioral, analyse→analyze) |
| justfile | Adds typos and typos-fix commands for running spell checker and auto-fixing typos |
| flake.nix | Enables typos in treefmt configuration and adds typos and typos-lsp to development shell packages |
| .github/workflows/ci.yml | Adds typos CI job that runs spell checker using Nix development environment |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 6 files
- Integrate main branch updates (rules structure, workflow updates) - Resolve justfile conflict by combining typos and build commands
|
thanks @jrebocho i got spell error lol omg |
Summary
typos.toml)typosandtypos-lspto flake.nix devShelljust typosandjust typos-fixcommandsTest plan
just typoslocally to verify spell checking worksSummary by cubic
Add a typos spell checker with en-us locale and enforce it in CI to catch spelling issues early. Standardizes US English across docs and comments.
New Features
Bug Fixes
Written for commit 8fc455c. Summary will update automatically on new commits.