New flake8-bugbear check B034: count/maxsplit/flag should be passed as keyword parameters

[jakkdl]

I added a new check to flake8-bugbear, and it would be great to have it incorporated into ruff as well. Copying from the issue PyCQA/flake8-bugbear#397

Context

• re.sub() doesn't substitute for all words with the replacement in the string when re.IGNORECASE is used and counter not specified python/cpython#99918 (comment)
• re.sub confusion between count and flags args python/cpython#56166

Example code

$ re.sub('a', 'b', 'aaa')
'bbb'
$ re.sub('a', 'b', 'aaa', re.IGNORECASE)
'bba'
>>> re.split(' ', 'a a a a')
['a', 'a', 'a', 'a']
>>> re.split(' ', 'a a a a', re.I)
['a', 'a', 'a a']
>>> re.split(' ', 'a a a a', flags=re.I)
['a', 'a', 'a', 'a']

The fourth parameter to re.sub[n] is in fact count, and the third to split is maxsplit, and as seen in the above linked issues this is a very common mistake.

Links

See the issue PyCQA/flake8-bugbear#397 for discussion around implementation, and PyCQA/flake8-bugbear#398 for the PR I wrote implementing it.

[tjkuson]

I believe this was implemented last night! #5669

It should be in version 0.0.278.

[jakkdl]

Whoa, so fast! I didn't even consider checking whether it was already implemented, I'll make sure to do that next time. Thanks! 🚀 🚀