Skip to content

fix: drop the unused typos word exemption - #118

Merged
michen00 merged 2 commits into
mainfrom
fix/typos-ignore-words-cleanup
Sep 23, 2026
Merged

michen00 merged 2 commits into
mainfrom
fix/typos-ignore-words-cleanup

Conversation

@michen00

@michen00 michen00 commented Sep 23, 2026 •

Copy link
Copy Markdown
Owner

What moved

arange (presumably meant for numpy.arange) doesn't appear anywhere in this tree — verified by stripping the entry and rerunning typos .: clean. Removed rather than documented, since there's nothing left to protect; add it back with the reason if a future change actually calls numpy.arange.

🤖 Generated with Claude Code

`arange` does not appear anywhere in this tree -- not as a bare
word, not as part of a numpy call. Nothing is protecting today;
add it back with the reason if a future change actually needs it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 23, 2026 19:49
@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Remove unused arange typo exemption

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Removes the unused arange exemption from typo-checking configuration.
• Restores typo detection for future accidental uses of the word.
High-Level Assessment

Removing the unused exemption is optimal because arange does not occur in the repository. Retaining or documenting the suppression would weaken typo checking without protecting any current usage.

Files changed (1) +0 / -1

Other (1) +0 / -1
_typos.tomlRemove the unused 'arange' ignore rule +0/-1

Remove the unused 'arange' ignore rule

• Deletes the 'arange' word exemption so the typo checker no longer suppresses a term unused by the repository.

_typos.toml

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The removed exemption is unused (no arange occurrences), and the change is a safe, localized config cleanup.

Review effort: Lite
Findings: None

What changed in this PR

This PR cleans up the typos configuration by removing an unused word-exemption entry that no longer protects anything in this repository.

Changes:

  • Remove the arange entry from extend-ignore-words-re in _typos.toml.
File Description
_typos.toml Removes an unused typos ignore-word exemption to keep spellchecking config minimal and accurate.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can reply 'qodo' on any finding to push back, ask questions, or dig deeper

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@michen00
michen00 merged commit ca14920 into main Sep 23, 2026
11 checks passed
@michen00
michen00 deleted the fix/typos-ignore-words-cleanup branch September 23, 2026 20:31
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.

2 participants