Add exception to WPS226 to allow commas and dots

[0anton]

Rule request

I suggest adding comma , and dot . to the list of allowed exceptions for WPS226

Thesis

Forcing defining a constant for dot and comma strings worsen the code readability without need.

Reasoning

It is a classical pattern to build a sentence with the code like:

strings = ['a', 'b', 'c']
','.join(strings)

It feels unnatural to define a constant (e.g. DOT, COMMA) for such obvious case.

Besides that the check WPS226 looks good. I don't like disabling it or increasing the number of string occurrences.
This extends the idea in #1534

[sobolevn]

@0anton do you want to send a PR with the fix? 🙂

[0anton]

Yes, sure! Just give me a couple of days for that.

…

On Mon 4. Oct 2021 at 19:53, Nikita Sobolev ***@***.***> wrote: @0anton <https://github.com/0anton> do you want to send a PR with the fix? 🙂 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2209 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEECBZEBM3FDPHFRAOIXDYTUFHSYHANCNFSM5FJ4TYBQ> .

[0anton]

@sobolevn please have a look at my suggestion #2210