Skip to content

Place project-specific settings first in codespell configuration #146

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 1 commit into from
Aug 18, 2021
Merged

Place project-specific settings first in codespell configuration #146

merged 1 commit into from
Aug 18, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 16, 2021

The ignore-words-list and skip settings of the codespell configuration file may required project-specific adjustments to fix false positives or avoid positives from externally maintained files. The other settings are universal. It will be convenient to have the settings the user might need to adjust at the place of highest visibility in the configuration file.

@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Aug 16, 2021
@per1234 per1234 requested a review from umbynos August 16, 2021 03:02
The `ignore-words-list` and `skip` settings of the codespell configuration file may required project-specific adjustments
to fix false positives or avoid positives from externally maintained files. The other settings are universal. It will be
convenient to have the settings the user might need to adjust at the place of highest visibility in the configuration
file.
@per1234 per1234 merged commit b42989c into arduino:main Aug 18, 2021
@per1234 per1234 deleted the codespell-config-order branch August 18, 2021 08:52
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants