Skip to content

Conversation

@owenleonard
Copy link

Here's a solution for Issue #110. I don't know if you want to head down this direction of adding options for scss-lint settings. I know there are other scss-lint settings I'd like to bring over to SassBeautify, and doing each one at a time could complicate things rather quickly.

I also added a unit test, but I'm having trouble running the unit tests so I'm not 100% sure it will pass (although it looks like it should)

I should also mention I left the current behavior as the default.

@owenleonard
Copy link
Author

I figured out how to run the unit tests, so I verified they are passing.

@Romainpetit
Copy link

Yes please ! 🐑

@owenleonard
Copy link
Author

Any word on these pull requests? Even a rejection is better than silence.

@bobbwal
Copy link

bobbwal commented May 12, 2017

Would be nice to have this in.

@attenzione
Copy link

did you tested for

.selector { 
  content: 'some text with 0.5$ inside';
}

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants