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

Add option to trim leading zero from numeric values with a decimal point #114

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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