Conversation
…s in test projects
| @@ -1,4 +1,7 @@ | |||
| [*.cs] | |||
| spelling_languages = en-us,en-gb | |||
| spelling_exclusion_path = ../exclusion.dic | |||
There was a problem hiding this comment.
the file this is pointing to has only 2 words, both already included in the top-level file. Is it an artifact of earlier changes?
| spelling_exclusion_path = ../exclusion.dic | |
| spelling_exclusion_path = ../../exclusion.dic |
There was a problem hiding this comment.
this is the (redundant?) file I am talking about
There was a problem hiding this comment.
Another behavior (bug) in Visual Studio spellchecker. There were some times when it would honor the exclusion.dic file in the root and other times when it would only honor the one in the immediate same folder as the project/.editorconfig. From your fix, its possible I had my path wrong. Either way, I got tired of troubleshooting MS/VS issues and in the end tried to focus on the point of the PR which was to fix a few spelling errors, not necessarily build a spellchecking tool pipeline.
See #1269 for some relevant discussion
List of changes
exclusion.dicfile for spellchecker words to ignore.editorconfig(s)to configure VS spellchecker for the library and unit test projectsTesting
All existing unit tests continue to pass.
Impact Analysis
None. No functional changes or public-facing APIs have been changed.