Skip to content

Expand codespell file exclusions to cover more generated files #168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 4, 2025

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Jun 4, 2025

The codespell tool is used to check for the presence of commonly misspelled words in the project files. Since we don't have control over their content, automatically generated files should be excluded from such checks for efficiency and to avoid false positives.

per1234 added 2 commits June 4, 2025 02:17
…ple Go modules

The "codespell" tool is used to check for the presence of commonly misspelled words in the project files. Since we don't
have control over their content, automatically generated files should be excluded from such checks for efficiency and to
avoid false positives. This includes the `go.mod` and `go.sum` Go module metadata files.

Projects may contain multiple Go modules in subfolders of the repository, each with their own metadata files.

Previously, the codespell configuration only excluded `go.mod` and `go.sum` in the root of the repository.
The "codespell" tool is used to check for the presence of commonly misspelled words in the project files. Automatically
generated files should be excluded from such checks for efficiency and to avoid false positives.

Although they are not checked into the repository, when contributors build the project the binary files will be present
in the project folder and thus subject to such checks. "codespell" is smart enough to automatically detect binary files
and not check their content, but it will be best to explicitly configure it to skip the files entirely for the sake of
efficiency.
@per1234 per1234 self-assigned this Jun 4, 2025
@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Jun 4, 2025
@per1234 per1234 merged commit 51db52c into arduino:master Jun 4, 2025
5 checks passed
@per1234 per1234 deleted the codespell-skip branch June 4, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant