Sass-lint dies when it encounters variables starting with $--

[TheJaredWilcurt]

Sass-Lint Version: 1.12.1

Code:

$--background-color: #F00

CLI Error:

src/sass/_var.sass

║ Line     │ Column   │ Type     │ Message                                                │ Rule ID              ║
╟──────────┼──────────┼──────────┼────────────────────────────────────────────────────────┼──────────────────────╢
║ 1        │ 1        │ error    │ Please check validity of the block starting from       │ Fatal                ║
║          │          │          │ line #1                                                │                      ║

---

I think naming your variable this way is ugly and terrible, but I'm just trying to import in the variables file used by ElementUI and that's how they name it, and that causes the linter to crash.

• https://github.com/ElemeFE/element/blob/dev/packages/theme-chalk/src/common/var.scss

[PurpleBabar]

Hi :)

Just making a guess for the moment but it seems to be a duplicate with #877 since the issue comes from -- and not espacially from $--
Could you just do a test where you try to remove the first $ in your import and tell us if the error moves on to another line or stays on this one ? :)

Thanks :)

[DanPurdy]

It'll be to do with the AST gonzales-pe, those errors come from that package. It's an old version running off a fork I'd done to fix some.issues as Gonzales hadn't been updated in a while.

[TheJaredWilcurt]

@PurpleBabar No I can't, because ALL Sass variables begin with a $.

$--red: #F00

body
    background: $--red

[PurpleBabar]

@DanPurdy yup :)
@TheJaredWilcurt i know that vars start with $ :). That was just to confirm that, in your case, the parser broke on -- and not on $--. If you can't test it, just wait for the update of the parser :)

Cheerz :)

[TheJaredWilcurt]

tonyganch/gonzales-pe#197

[webartov]

@PurpleBabar
The following doesn't call an error on v. 1.12.1 (but maybe I misunderstood the question)

$test: #0f0;
.element {
  --main-bg-color: $test;
}

And this one calls:

$--test: #0f0;
.element {
  color: $--test;
}

Basically it's needed for https://element.eleme.io
By default all variables there come as $--
Please also note that $- or $-var-------------test will not call an error too.
@DanPurdy
Could you please tell if there are any tricks to avoid the error or fix will come soon?
We badly need this for our project and I don't want to switch to scss-lint just because of this.

[webartov]

Looks like dead to me: tonyganch/gonzales-pe#290 or not tonyganch/gonzales-pe#290 (comment)

@DanPurdy will the patch solve the issue with $-- too?

[webartov]

Can somebody help apply the fix to the linter?
tonyganch/gonzales-pe#295

[TheJaredWilcurt]

So it's been a year. looks like gonzales might have fixed it on their end, but never did a release, and it looks like this project is marked as deprecated. lame

[webartov]

So it's been a year. looks like gonzales might have fixed it on their end, but never did a release, and it looks like this project is marked as deprecated. lame

I think gonzales fixed only part of the issue. please refer to my comment here for details tonyganch/gonzales-pe#295 (comment)

[TheJaredWilcurt]

https://www.npmjs.com/package/gonzales-pe

Gonzales-pe v4.3.0 should fix this. Just need to update that dependency and do a new release