blanklineBetweenSelectors has a failing case about sassbeautify HOT 5 CLOSED

That regular expressions is flaky. To be honest I'm very tempted to remove this feature. I merged this feature in with pull request #30 without properly testing it. IMO the only reliable way of formatting code is to create a lexer/parser which is totally outside the scope of this plugin.

from sassbeautify.

By the way, I ran into this while trying to use SassBeautify in conjunction with SublimeLinter-contrib-scss-lint. It has a rule for a single selector per line. I'm finding that sass-convert isn't very compatible with scss-lint.

Is the linter not configurable?

from sassbeautify.

It is configurable, but I'd rather make a beautifier/formatter match chosen
linting rules rather than the reverse. It may be that I need to write a
lexer/parser that can read in those linting rules and do the formatting
rather than relying on what sass-convert supplies...

On Thu, Feb 13, 2014 at 2:20 AM, Richard Willis [email protected]:

By the way, I ran into this while trying to use SassBeautify in
conjunction with SublimeLinter-contrib-scss-lint. It has a rule for a
single selector per line. I'm finding that sass-convert isn't very
compatible with scss-lint.

Is the linter not configurable?

Reply to this email directly or view it on GitHubhttps://github.com//issues/37#issuecomment-34960776
.

from sassbeautify.

I'd rather make a beautifier/formatter match chosen linting rules rather than the reverse

I agree. Really this plugin only provides the flexibility of sass-convert, which is not a lot! I have been thinking about providing post-sass-convert formatting features, but I'm still undecided. Perhaps there's a solid python sass lexer out there that I can use, but saying that it doesn't seem right to do it in python land, seeing as though sass already has a lexer/parser. If I was more familiar with ruby I would explore the option of creating a custom gem that uses sass but also provides additional formatting options.

from sassbeautify.

I have decided to remove this feature entirely.

from sassbeautify.