Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Dec 29, 2025

Remove typos configuration and packages from the Python SDK.

Changes:

  • Remove typos.enable and typos.configFile from treefmt.programs in flake.nix
  • Delete typos.toml configuration file
  • Remove typos and typos-lsp from devShells buildInputs

The typos tool is no longer needed for this project's workflow.


Summary by cubic

Removed the typos CLI and its config from the Python SDK to simplify the dev environment and flake setup.

  • Refactors
    • Removed typos.enable and typos.configFile from treefmt.programs in flake.nix.
    • Deleted typos.toml.
    • Dropped typos and typos-lsp from devShells buildInputs.

Written for commit 13a7dc9. Summary will update automatically on new commits.

Remove typos configuration and packages:
- Remove typos.enable and typos.configFile from treefmt.programs
- Delete typos.toml configuration file
- Remove typos and typos-lsp from devShells buildInputs

The typos tool is no longer needed for this project's workflow.
Copilot AI review requested due to automatic review settings December 29, 2025 08:43
@ryoppippi ryoppippi enabled auto-merge (squash) December 29, 2025 08:43
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 2 files

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 removes the typos CLI tooling from the Python SDK project's development environment, simplifying the tooling stack by eliminating an unused spell-checking tool.

  • Removes typos configuration from the Nix flake's treefmt programs
  • Deletes the typos.toml configuration file
  • Removes typos and typos-lsp packages from the development shell

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
typos.toml Completely removed the typos configuration file including locale settings and file exclusion patterns
flake.nix Removed typos integration from treefmt.programs and removed typos/typos-lsp packages from devShells buildInputs

I've completed my review of this pull request. The changes are clean and complete - all references to the typos tooling have been properly removed from both the configuration file and the Nix flake. There are no remaining references to typos in the codebase, and the removal is consistent across all locations where it was configured. No issues were identified.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@glebedel glebedel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryoppippi ryoppippi merged commit fa1fb7f into main Dec 29, 2025
21 checks passed
@ryoppippi ryoppippi deleted the chore/remove-typos-cli branch December 29, 2025 10:29
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