yarikoptic opened a new pull request, #8761: URL: https://github.com/apache/incubator-devlake/pull/8761
Add [codespell](https://github.com/codespell-project/codespell) configuration and fix existing typos. I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback. CI workflow has `permissions` set only to `read` so should be safe. ## Changes ### Configuration & Infrastructure - Added `.codespellrc` with comprehensive skip patterns for caches (`.cache`, `.npm`, `.yarn`), test fixtures (`*/e2e/raw_tables/*`, `*/e2e/snapshot_tables/*`), lock files, SVGs, CSS - Created GitHub Actions workflow to check spelling on push and PRs - camelCase/PascalCase ignore-regex to avoid flagging Go/TypeScript identifiers (`HasTable`, `OrderIn`, `repoRes`, etc.) ### Domain-Specific Whitelist Added legitimate terms that codespell flags as typos: - `convertor`/`convertors` - project's deliberate spelling convention for Go types and filenames (`DataConvertor`, `*_convertor.go`) - `crypted` - variable name in AES encrypt/decrypt functions - `te` - Tapd API field name - `thur` - Thursday abbreviation in Grafana dashboard SQL ### Typo Fixes **Ambiguous typos fixed manually** (11 distinct typos across 20 files): - `convered` -> `converted` (jenkins build model comments) - `covert` -> `converted` (issue_status_history log message) - `Colume` -> `Column` (migrationhelper error messages) - `poping` -> `popping` (api_client comment) - `limt` -> `limit` (6 connection model struct tag comments) - `faile` -> `failed` (pipeline error messages) - `ths` -> `the` (service.go comment) - `reloades` -> `reloads` (blueprint.go comment) - `verifi` -> `verify` (job_test.go comment) - `Troughput` -> `Throughput` (Grafana dashboard display text) - `actived`/`activedColor` -> `activated`/`activatedColor` (onboard styled components) **Non-ambiguous typos fixed via `codespell -w`** (67 fixes across 47 files): Common fixes include: `doens't` -> `doesn't`, `faild` -> `failed`, `Internel` -> `Internal`, `mintue` -> `minute`, `mulitple`/`mutiple`/`muliple` -> `multiple`, `pipline`/`pipleline`/`pipepline` -> `pipeline`, `envioronment` -> `environment`, `Reveiw` -> `Review`, `serverity` -> `severity` ### Historical Context This project has had 119 prior commits fixing typos manually, demonstrating the value of automated spell-checking. ## Testing Codespell passes with zero errors after all fixes. --- Generated with [Claude Code](https://claude.com/claude-code) and love to typos free code -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
