Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Dec 18, 2025

Summary

  • Add typos spell checker configuration (typos.toml)
  • Add typos CI job to workflow
  • Add typos and typos-lsp to flake.nix devShell
  • Enable typos in treefmt
  • Add just typos and just typos-fix commands
  • Fix existing typos (UK to US English spelling)

Test plan

  • Run just typos locally to verify spell checking works
  • Verify CI workflow runs typos check

Summary 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

    • Added typos.toml (en-us, ignore patterns, excludes).
    • CI workflow now runs typos; enabled in treefmt.
    • Nix devShell includes typos and typos-lsp; added just typos and just typos-fix commands.
  • Bug Fixes

    • Converted UK → US spellings in README and comments.

Written for commit 8fc455c. Summary will update automatically on new commits.

- 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
Copilot AI review requested due to automatic review settings December 18, 2025 19:29
Copy link
Contributor

Copilot AI left a 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.toml configuration 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.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a 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
@ryoppippi ryoppippi enabled auto-merge (squash) December 18, 2025 19:36
@ryoppippi ryoppippi merged commit 14f7eed into main Dec 18, 2025
6 of 9 checks passed
@ryoppippi ryoppippi deleted the feat/add-typos-checker branch December 18, 2025 19:36
@ryoppippi
Copy link
Member Author

thanks @jrebocho i got spell error lol omg

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants