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

Bug: Indentation rule with parens #46

Closed
bgriffith opened this issue Aug 29, 2015 · 4 comments
Closed

Bug: Indentation rule with parens #46

bgriffith opened this issue Aug 29, 2015 · 4 comments

Comments

@bgriffith
Copy link
Member

Using parens causes an issue with the indentation rule.

// Causes an error - 'Indentation of 2, expected 4'
@if (1 == 1) {
  content: 'foo';
}

// Is ok
@if 1 == 1 {
  content: 'foo';
}

// Is ok
@mixin bar($foo) {
  content: 'bar';
}

Using v1.0.0

@Snugug
Copy link
Member

Snugug commented Aug 29, 2015

Boo! This comes from the work to allow proper indentation for maps. Let's squash.

@Snugug
Copy link
Member

Snugug commented Aug 29, 2015

Where is the error coming from? content?

@bgriffith
Copy link
Member Author

Yep, here's the error.
screen shot 2015-08-29 at 22 19 33
screen shot 2015-08-29 at 22 19 54

@Snugug Snugug closed this as completed in b4ebef0 Aug 29, 2015
@Snugug
Copy link
Member

Snugug commented Aug 29, 2015

I'm going to see if anything else pops up this weekend and release a patch fix by the end of the weekend

@Snugug Snugug added this to the 1.0.1 milestone Aug 29, 2015
donabrams pushed a commit to donabrams/sass-lint that referenced this issue Nov 19, 2015
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

2 participants