connectedhomeip
connectedhomeip copied to clipboard
[Tooling] update pyspelling in SDK to support camel-case
Reproduction steps / Feature
newer versions fixed the camel-case option to aspell. The spec repo uses this version. We should update the SDK to do the same. Then we can remove some of the excluded files from the spell checker.
Platform
python
Platform Version(s)
No response
Type
CI tested
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response
It's also unclear why we have two spell checkers running - it's probably worthwhile to consolidate these.
After updating, we should also prune the wordlist to exclude the things that were added to work around not having camel case. When we did this in the spec, we found lots of spelling errors.
Pyspelling updated, Reviewdog commented out Slowly removing camelcase entries from the wordlist - regex indicates there are a few hundred