Add codespell support with configuration and fixes#1365
Open
yarikoptic wants to merge 8 commits intogrobidOrg:masterfrom
Open
Add codespell support with configuration and fixes#1365yarikoptic wants to merge 8 commits intogrobidOrg:masterfrom
yarikoptic wants to merge 8 commits intogrobidOrg:masterfrom
Conversation
- Fix 'splitted' to 'split' in comment - Add inline codespell:ignore for regex patterns that may intentionally match OCR typos
- Add ignore list: consol, countr, inpu, currenty, currentx - Fix 'inputed' to 'inputted' in javadoc comment - Add .npm/ to .gitignore to prevent npm cache commits
- Add to ignore list: countr, inpu, currenty, currentx - Fix 'inputed' to 'inputted' in javadoc comment
Documentation fixes: - prefered choise → preferred choice - for nore details → for more details - table is maked → table is marked - thei consequences → their consequences Code fixes: - limted → limited - extact → exact (3 occurrences) - token positons → token positions - preceeding → preceding - wither → either - skipt → skipped
Applied codespell -w to automatically fix clear typos across 81 files: - Documentation files (29 files) - Java source files (52 files) Common fixes include: - occured → occurred - instanciate/instanciated → instantiate/instantiated - immediatly → immediately - prefered → preferred - orginal → original - bellow → below - recommand/recommanded → recommend/recommended - intials → initials And many others
- Runs on push to master and on pull requests - Uses official codespell-project/actions-codespell@v2 - Checks both file contents and filenames - Read-only permissions for security
Revert Acknowledge?meant? → Acknowledge?ment? in regex pattern. The pattern matches 'Acknowledgment' or 'Acknowledgement', not 'Acknowledgemeant'. Added inline codespell:ignore comment to protect regex pattern.
Member
|
Thanks @yarikoptic. I will merge this in sync with #1358 which may be in contrast between each other |
Author
|
oh -- likely do not bother merging! I should be able to easily redo fixes. Just proceed with refactoring first and then let's fix those typos up |
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
2 participants
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.
Changes
Configuration & Infrastructure
.codespellrcconfiguration with comprehensive skip patterns.npm/to.gitignoreDomain-Specific Whitelist
Added legitimate terms that codespell flags as typos:
serie- French word for "series"blockin- ML feature labelpunctuations,ther,ist,usre,theses,ue- domain-specific termsconsol- abbreviation for "consolidation"countr,inpu- variable namescurrenty,currentx- coordinate variable typos (currentY, currentX)Typo Fixes
Ambiguous typos fixed manually (12 fixes with context review):
prefered choise→preferred choice,nore→more,maked→marked,thei→theirlimted→limited,extact→exact(3×),positons→positions,preceeding→preceding,wither→either,skipt→skippedNon-ambiguous typos fixed automatically (221 fixes in 80 files):
Common fixes include
occurred,instantiate,immediately,preferred,original,below,recommend, and many others.Regex Pattern Protection
Added inline
codespell:ignorecomments for:ddresspatterns (intentionally match OCR typos in source text)mentpattern (matches "Acknowledgment"/"Acknowledgement")Testing
✅ Codespell passes with zero errors after all fixes
🤖 Generated with Claude Code