no-duplicate-dollar-variables doesn't respect scopes

no-duplicate-dollar-variables rule reports variables with the same name but declared in different scopes. The snippet below produces error Unexpected duplicate dollar variable $padding-x scss/no-duplicate-dollar-variables.

.FormControl-label {
  $padding-x: $padding-small-x;

  @include input-overlay;

  position: relative;
  top: 0.40em;
  margin-left: $margin-medium-x;
  padding-left: $padding-x;
  padding-right: $padding-x;
  font-size: $font-size-small;
}


.FormControl-input {
  $padding-x: $padding-medium-x;

  width: 100%;
  border: 1px solid $color-grey-light;
  border-radius: 0;
  padding-left: $padding-x;
  padding-right: $padding-x;
  height: $row-height;
}

For me it’s pretty obvious that this shouldn’t be reported. Am I wrong?

[kristerkari]

@chrismbarr those are not actually false positives, to avoid that you can use the ignore options of the rule:
https://github.com/kristerkari/stylelint-scss/blob/master/src/rules/no-duplicate-dollar-variables/README.md#ignoreinsideatrules-array-of-at-rules

[masi]

WIth 3.18 the rule triggers:

.foobar {
  $gapWidth: 5px;

  @media (min-width: 600px) {
    $gapWidth: 15px;
  }
}

I had assumed that the media query defines a scope.