-
Notifications
You must be signed in to change notification settings - Fork 534
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
Stop property-sort-order
and no-value-vendor-prefix
from throwing with css variables
#1225
base: develop
Are you sure you want to change the base?
Conversation
These cause the tests to throw.
If we want to see if Usage exists, it needs an indexOf !== -1, not > 0
property-sort-order
from throwing with css variablesproperty-sort-order
and no-value-vendor-prefix
from throwing with css variables
Please merge this ASAP. This is blocking me from integrating sass-lint on our workflow... |
I would also like this merged :) |
@justinanastos there is a merge conflict. @DanPurdy is there additional effort or comments to be made on this PR beyond the merge conflict? |
@@ -1,4 +1,5 @@ | |||
.foo | |||
--css-variable: 100vh; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--css-variable: 100vh; | |
--css-variable: 100vh |
Is there anything holding this up from being merged other than the conflict? |
@@ -99,8 +99,14 @@ module.exports = { | |||
|
|||
if (block) { | |||
block.forEach('declaration', function (dec) { | |||
var prop = dec.first('property'), | |||
name = prop.first('ident'); | |||
var prop = dec.first('property'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
var prop = dec.first('property'); | |
var prop = dec.first('property') || dec.first('customProperty'); |
from #1207
return; | ||
} | ||
|
||
var name = prop.first('ident'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
var name = prop.first('ident'); | |
var name; | |
if (typeof(prop.first) === 'function') { | |
name = prop.first('ident'); | |
} |
What do the changes you have made achieve?
Are there any new warning messages?
Have you written tests?
Have you included relevant documentation
Which issues does this resolve?
Fixes #1224
DCO 1.1 Signed-off-by: Justin Anastos <justin.anastos@gmail.com>