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

[sass] Dies on empty class declaration #456

Open
dempfi opened this issue Dec 17, 2015 · 4 comments
Open

[sass] Dies on empty class declaration #456

dempfi opened this issue Dec 17, 2015 · 4 comments

Comments

@dempfi
Copy link

dempfi commented Dec 17, 2015

I get error

.../node_modules/sass-lint/lib/groot.js:21
    throw new Error('Parsing error' + fileInfo + ': ' + e.message);
    ^

with empty claass declarations

.some-class

node v5.2.0, npm 3.3.12, sass-lint 1.4.0

@DanPurdy
Copy link
Member

This is an issue with gonzales-pe our AST im afraid. We're awaiting updates as we speak.

@dempfi
Copy link
Author

dempfi commented Dec 17, 2015

thanks for reply @DanPurdy. On gonzaless-pe repo last commit was 2 month ago and there is many issues on sass-lint where it mentioned. Do you plan to move from it to another AST parser? And how I can help solving this issues with AST?

@DanPurdy
Copy link
Member

So the developer of gonzales-pe has been busy recently but there will hopefully be some updates soon.

Most of the issues we have are sorted in the latest version of gonzales-pe but unfortunately it's not quite stable enough for us to upgrade yet.

AST's that support sass and scss are hard to come by but there are some promising plans for new ones..

If you'd like to contribute to gonzales-pe I know the developer there would be more than happy for the assistance, I would myself but time is very limited at the moment..

@bgriffith
Copy link
Member

Out of interest are you running ruby sass or libsass? Libsass (3.2.5 at least - sassmeiser) doesn't like empty sass class declarations.

@Snugug Snugug added this to the 1.5.1 - Latest Gonzales milestone Feb 7, 2016
@DanPurdy DanPurdy removed this from the 1.6.0 - Latest Gonzales milestone Apr 22, 2016
@bgriffith bgriffith changed the title Dies on empty class declaration [sass] Dies on empty class declaration Aug 28, 2017
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

4 participants