Skip to content

Feat: Added Spell-Checker and fixed spellings#208

Open
GunterSchmidt wants to merge 1 commit intouutils:mainfrom
GunterSchmidt:add_cspell_project_dictionaries
Open

Feat: Added Spell-Checker and fixed spellings#208
GunterSchmidt wants to merge 1 commit intouutils:mainfrom
GunterSchmidt:add_cspell_project_dictionaries

Conversation

@GunterSchmidt
Copy link
Copy Markdown
Contributor

For Code Spell Checker extension.

Fixed some clippy warnings.

For Code Spell Checker extension.

Fixed some clippy warnings.
@GunterSchmidt GunterSchmidt force-pushed the add_cspell_project_dictionaries branch from af533ab to 2646457 Compare April 12, 2026 21:24
@GunterSchmidt
Copy link
Copy Markdown
Contributor Author

@sylvestre Could you take a look at this? I then can continue

Copy link
Copy Markdown

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes require a very different level of review attention. So when you have clippy fixes sneaking in with much larger spell check changes, now all changes require a high-level of review attention and workload - cause you can't be sure.

The commit message and title should mention that the spell checker configuration are only usable by VSCode (and not run in CI either).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants