Sass Mixed Declarations new change causing Unexpected duplicate selector

Rules and Languages Report False-positive / False-negative...
1

Hey,

Because of the recent changes to Sass see here: Sass: Breaking Change: Mixed Declarations I’ve had to change the following code:

td,
  th {
    @include small-text;
    padding: 12px 15px;
    vertical-align: top;
    min-width: 300px;
    width: 300px;
  }

to this:

td,
  th {
    @include small-text;
    & {
      padding: 12px 15px;
      vertical-align: top;
      min-width: 300px;
      width: 300px;
    }
  }

This is resulting in SonarQube giving a maintainability issue:
[Unexpected duplicate selector ".table td, .table th"]
Has SonarQube just not adjusted for this or am I doing something wrong?

4

Hello @Reskun,

thanks for reporting this. I created a ticket to track this bug.

Cheers,
Victor

8

Hello everyone,

Is there any news on this topic? In which Sonarqube version can we expect the rule to be adjusted?

thx

9

Hi @markluebbehuesen,

The ticket Victor linked to is still Open.

 
Ann

10

Hi @ganncamp ,

thanks, I see that there is a ticket for this topic and that several people have already looked into it.

However, I can’t say whether there is already a plan for the update or if it’s scheduled for a future update.

Can you provide more details or find out?

thanks,

Mark

11

Hi Mark,

Sorry, I don’t have any further details.

 
Ann