feat(lint): adjust LESS whitespaces to match prettier format

[mvorisek]

This PR fixes whitespaces unfixed/undetected in GH-2610 and major whitespace changes required for future Prettier integration.

Double quotes and lowecased hex colors are required by Prettier - prettier/prettier#5158 - and there is no config to change it. As long as we want to use Prettier, we must accept opinionated formatting by Prettier.

[mvorisek]

Let's land these few CS changes now, PR #2611 needs some Prettier tool fixes, so PR #2611 cannot be finished soon.

And good news! With this PR, the era of 10k LOC changed daily is done. For JS and LESS. Thank you @lubber-de ❤️

[mvorisek]

To verify this PR I run build before all changes and after. Then I run Prettier on both data (Prettier seems quite safe in sense of corrupting AST or comments).

There are only a few changes in semantic.css.min like:

will selectors with removed space before :: match always the same elements?

I have also double checked all rtl: comments (#2612 (comment)) and I can confirm none of them were removed after first LESS CS PR (e589cd1)

[lubber-de]

To verify this PR I run build before all changes and after. Then I run Prettier on both data (Prettier seems quite safe in sense of corrupting AST or comments).

will selectors with removed space before :: match always the same elements?

No, those would have a different meaning. the space indicates a child node, and without the space it indicates the exact node

Damn, this is something that has been removed by the whitespace changes, which i did not recognize.... This has to be reverted to the state it was in 2.9.0

[lubber-de]

The wrong removement was done by fc431e8

[mvorisek]

I marked this PR as a draft and will take care of it.

[mvorisek]

The change of removed space before pseudo classes is fixed/reverted.

[lubber-de]

Thank you, LGTM

[lubber-de]

@mvorisek
When i run the lint locally, stylelint moans about the "each" statement... but the CI report does not do this.
Any idea what i am doing differently (yes, i ran the additional yarn packages from the ci as well to get the additional packages and i tried node 16 and 18.

[mvorisek]

Did you installed all the packages that CI installs? It seems like the CSS is parsed without the less preprocessor. Maybe some config issue, you call the command from different pwd...