Skip to content

Conversation

@per1234
Copy link
Contributor

@per1234 per1234 commented Sep 16, 2025

The assets include infrastructure to check for commonly misspelled words in the project files.

Although the system is designed to reduce the incidence of false positives, they are still relatively common. For this reason, it is important to clearly communicate to the contributor how such false positives can be resolved. This can be accomplished effectively by printing instructions when the spell check fails.

In order to avoid excessive verbosity of the output (note that the failure could just as well be a true positive), the description of the procedure is maintained in the asset documentation and the output message simply provides the URL of that documentation.

The assets include infrastructure to check for commonly misspelled words in the project files.

Although the system is designed to reduce the incidence of false positives, they are still relatively common. For this
reason, it is important to clearly communicate to the contributor how such false positives can be resolved. This can be
accomplished effectively by printing instructions when the spell check fails.

In order to avoid excessive verbosity of the output (note that the failure could just as well be a true positive), the
description of the procedure is maintained in the asset documentation and the output message simply provides the URL of
that documentation.
@per1234 per1234 self-assigned this Sep 16, 2025
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Sep 16, 2025
@per1234 per1234 merged commit d0e5e63 into arduino:main Sep 16, 2025
51 checks passed
@per1234 per1234 deleted the spell-check-instructions branch September 16, 2025 03:07
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.

1 participant