Add 'dist' to codespell skip list for codespell#39
Conversation
this is the location to tune codespell config to make it green again.
🔍 PR PreviewPreview URL: https://ReproNim.github.io/ReproInventory/pr-previews/pr-39/ Last updated: Thu, 23 Apr 2026 17:35:25 GMT |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .codespellrc configuration to exclude the dist directory from spell checking. The feedback suggests using a glob pattern like **/dist to correctly target build artifacts in subdirectories and recommends adding **/node_modules to the skip list to improve performance and avoid false positives.
| [codespell] | ||
| # Ref: https://github.com/codespell-project/codespell#using-a-config-file | ||
| skip = .git*,*.svg,package-lock.json,*.css,.codespellrc | ||
| skip = .git*,*.svg,package-lock.json,*.css,.codespellrc,dist |
There was a problem hiding this comment.
The pattern dist only matches at the root level. Since the project contains a frontend directory, the build output (likely frontend/dist) will not be skipped by this pattern. Using **/dist is more effective for ignoring build artifacts in subdirectories.
Additionally, adding **/node_modules to the skip list is strongly recommended to improve performance and avoid false positives from third-party code, which is especially relevant given that check-hidden is set to true.
skip = .git*,*.svg,package-lock.json,*.css,.codespellrc,**/dist,**/node_modules
2 participants
this is the location to tune codespell config to make it green again.