Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parsing error with interpolated expression #321

Closed
jpzwarte opened this issue Oct 17, 2015 · 20 comments
Closed

Parsing error with interpolated expression #321

jpzwarte opened this issue Oct 17, 2015 · 20 comments

Comments

@jpzwarte
Copy link

left: calc(50% - #{$eb-hint-width / 2});

This causes a parser error to be thrown: "Please check validity of the block starting from line #40"

Using sass-lint-1.3.1

@DanPurdy
Copy link
Member

There's a problem with interpolation within the current version of the AST we use, it looks to be fixed in the new version though, once it's stable we'll be moving up to it.

Similar issue here #210

@micahgodbolt
Copy link
Member

Also found in #313

@DanPurdy
Copy link
Member

This is still causing an error in gonzales-pe 3.2.1 have logged an issue over there. You can find it here

Will keep you updated

@stowball
Copy link

Just voicing a +1 as it happened to me as soon as I tried to implement sass-lint :(

@DanPurdy
Copy link
Member

+1s here are kind of useless unfortunately as we're blocked by a dependency, it would be better for you to report this on the gonzales-pe repository :(

@stowball
Copy link

Noted!

@clcpolevaulter
Copy link

hey @DanPurdy or @bgriffith, do you anticipate a release date for the gonzales 3.2.6 fixes

@bgriffith
Copy link
Member

Hey @clcpolevaulter - we're just finishing off the remaining issues with the rules. I'd hope we could do a release by next week if not sooner.

@webgefrickel
Copy link

@bgriffith that would be very cool, since I am experiencing this issue as well, when trying to do sth. like:

#{$side}: $value;

@colin-marshall
Copy link

@webgefrickel good news, that was definitely fixed in 3.2.6. I put in an issue for it that was resolved a couple weeks back (tonyganch/gonzales-pe#140) so the forthcoming sass-lint 1.6.0 should resolve that.

@clcpolevaulter
Copy link

hey @DanPurdy or @bgriffith, we are really looking forward to these changes. Any updates on when the next release will be?

@colin-marshall
Copy link

@clcpolevaulter: @DanPurdy said earlier today that it will hopefully be ready by this weekend foundation/foundation-sites#7620 (comment)

@DanPurdy
Copy link
Member

Sorry guys I'm just really swamped with work at the moment but I'll definitely be looking to have this last rule updated and done very soon..

@clcpolevaulter
Copy link

@DanPurdy @bgriffith is there a branch in progress on this release? I'd love to help if needed.

@DanPurdy
Copy link
Member

DanPurdy commented Mar 9, 2016

@clcpolevaulter We're actually done, updated and ready to release 1.6, BUT the AST currently doesn't install on windows correctly so we can't really release it until that's sorted..

See the bottom of this issue tonyganch/gonzales-pe#149 for more info

@EightArmCode
Copy link

Any word on this?

@DanPurdy
Copy link
Member

@EightArmCode the majority of these issues should hopefully be fixed in our 1.6 update as it includes the latest version of the AST we've been using. I would expect it to come out in the next few days.. Hopefully..

@Regaddi
Copy link

Regaddi commented Apr 12, 2016

@DanPurdy Any progress on this? Really looking forward to it! 😃

@bgriffith
Copy link
Member

@jpzwarte, @Regaddi, @EightArmCode, @clcpolevaulter - This is fixed in 1.6

@DanPurdy
Copy link
Member

ping @colin-marshall too ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants