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
New rule: No non numeric dimensions #357
New rule: No non numeric dimensions #357
Conversation
@nex3 fyi |
It's probably also a good idea to test that this only applies to unquoted strings. |
good call on the test! added a test case for that. |
@kristerkari - ptal when you can! |
Sorry, I have been really busy for the last 2 weeks. I think that the new rules are okay, but only thing that remains is to think about the naming, are the rules names matching the stylelint naming conventions? Are the rules in the right "category"? etc. I will try to think about the names this weekend, and give suggestions for the names if I can come up with any. |
No worries! I appreciate all of your help and completely understand that you're busy. Thanks a lot! |
The old sass/linter had a rule to ensure that string interpolated values didn't have units attached to them (because they'd be strings instead of numbers).
This rule replicates that. The tests are the same for the sass/linter rule + this rule, so I have good confidence it'll give the same user experience.