Expected 1 space after colon with &:not(:first-child)

[Sental]

Preconditions

1. In a less file if you use &:not(:first-child) it generates spacing warnings.

Steps to reproduce

1. Use &:not(:first-child) in a less file.
2. Run the coding standard from command line. vendor/bin/phpcs --standard=Magento2 path/to/module

Expected result

1. No Warnings are generated.

Actual result

1. Warnings are generated 154 | WARNING | Expected 1 space after colon in style definition; newline found 154 | WARNING | Expected 1 space after colon in style definition; 0 found

[m2-assistant[bot]]

Hi @Sental. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this

---

• Join Magento Community Engineering Slack and ask your questions in #github channel.

[mrtuvn]

Same issue experiences. Actually all pseudo classes not need require spacing after

[hostep]

The phpcs tools should only scan *.php (and maybe *.phtml) files, why is it being used to scan *.less files as well? That makes little sense?

[ihor-sviziev]

@sivaschenko, could you take a look? We have an example where this issue reproducing on a PR - https://github.com/magento/magento2/pull/35101#issuecomment-1146916983

[sivaschenko]

@ihor-sviziev yes, we'll work on it

[svera]

@Sental @mrtuvn just released version 25 which should fix this issue.

[ihor-sviziev]

I guess the changes are the following: https://github.com/magento/magento-coding-standard/pull/405

@Sental, could you confirm that issue is solved now?

[mrtuvn]

Also found magento yelling warning non-sense at this code

.box-tocart {
    .paypal:first-of-type {
      margin-top: 13px;
    }
  }

:first-of-type totally valid. It's doesn't require space after that colon

.lib-link(
                @_link-color: @primary__color,
                @_link-text-decoration: none,
                @_link-color-visited: @primary__color,
                @_link-text-decoration-visited: none,
                @_link-color-hover: @primary__color,
                @_link-text-decoration-hover: underline,
                @_link-color-active: @primary__color,
                @_link-text-decoration-active: underline
            );

Also detected new line after colon at mixins less declare => still valid with me in my point of view