connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

[Tooling] update pyspelling in SDK to support camel-case

Open cecille opened this issue 1 year ago • 3 comments

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

cecille avatar Jan 22 '24 14:01 cecille

It's also unclear why we have two spell checkers running - it's probably worthwhile to consolidate these.

cecille avatar Jan 22 '24 14:01 cecille

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.

cecille avatar Jan 22 '24 14:01 cecille

Pyspelling updated, Reviewdog commented out Slowly removing camelcase entries from the wordlist - regex indicates there are a few hundred

kliao-csa avatar Feb 15 '24 23:02 kliao-csa