Consider re-adding "shouldnot" to allow dictionary

[jackfrancis]

Hi folks,

It seems that w/ today's v2.2.2 release the word "shouldnot" has been demoted to "not allowed by default". Because terms like Should and ShouldNot are rather common in test assertion libraries across the developer ecosystem, I wonder if other folks had to quickly update their ignore files today to unblock CI.

Here's one example of some usages in the golang ginkgo test framework:

• https://pkg.go.dev/github.com/onsi/gomega#AsyncAssertion

Thanks for consideration!

[peternewman]

Hi @jackfrancis ,

While it's clearly not a valid word, please feel free to open a PR to move it into the code dictionary, then it won't be enabled by default only if people request to scan with that dictionary (hopefully not on code).

[jackfrancis]

Thank you @DimitriPapadopoulos !

[DimitriPapadopoulos]

You're welcome and thank you for this contribution.