no-mergeable-selectors incorrectly trying to merge two selectors in different mixins

[bgriffith]

This rule incorrectly tries to merge two selectors within two different mixins.
SassLint: 1.3.1

@mixin foo {
    &:disabled {
        color: red;
    }
}

@mixin input-bar {
    &:disabled {
        cursor: blue;
    }
}

[DanPurdy]

Yeah, i need to look at mixins as part of #307 anyway so I'll combine the effort then

[bgriffith]

Ah yeah, I see they're related.

[brokentone]

Same issue here. Although a slightly different test case for me -- one inside a mixin.

    & a { 
      display: block;
    }   

    @include breakpoint($bp-760) {
      & a { 
        display: inline-block;
      }   
    }   

[DanPurdy]

Yeah, I'm hoping to look at this soon, it's going to be a little bit of a refactor it looks like.

[tonifisler]

Same issue here with media queries:

.spacing {
  margin: 16px 0;
  &.spacing-sm {margin: 8px 0;}
  @media only screen and (max-width: 767px) {
    margin: 8px 0;
    &.spacing-sm {margin: 5px 0;}
  }
}

.spacing-bottom {
  margin-bottom: 32px;
  &.spacing-sm {margin-bottom: 15px;}
  @media only screen and (max-width: 767px) {
    margin-bottom: 15px;
    &.spacing-sm {margin-bottom: 10px;} // sass-lint triggers here
    // and suggests merging with the rule in the media query of .spacing
  }
}

[DanPurdy]

Don't worry, this is in progress and should be resolved in our 1.6 update which we're hoping to get to you soon.

[DanPurdy]

@tonifisler @bgriffith @brokentone All of your examples are fixed in the rewrite of the mergeable selectors rule released with sass-lint 1.6.0!