Add flake8-implicit-str-concat check to Ruff

[josh-fell]

This check was enabled initially in #23873, but hasn't been part of the Ruff checks to-date. Let's add it!

[o-nikolas]

Other than one comment, looks good to me! Love this one ❤️

[potiuk]

Other than one comment, looks good to me! Love this one ❤️

Likewise :)

[pankajkoti]

nice, just wondering how those string concatenations existed in so many places :) maybe it was another pre-commit responsible for it.

[potiuk]

nice, just wondering how those string concatenations existed in so many places :) maybe it was another pre-commit responsible for it.

We used to get rid of them by flake I think - but at the time we moved to ruff it was not available as far as I remember.
And those strings are created by black reformatting - for example when you remove if and the whole block gets de-dented - the previously split strings (because they were too long) are moved to one line. But black (and ruff-black follows it) has the rule that it never EVER changes the AST so concatenating two strings is not possible in ruff.

That's the root cause we had so many of those.

And yes. I got far too many of those as well and it started to be annoying :)