You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It's not clear if rules not present in sass-lint.yml files are turned off automatically (ESLint 1.0 style) or not.
With early runs of gulp-sass-lint, that does not seem to be the case; for codebases that have both Sass & SCSS litn files, it seems there's issues propagating from gonzales-pe unless trailing-semicolon is explicitly set to 0 rather than merely omitting the rule altogether:
Seemingly any rule not listing in the sass-lint.yml should be turned off (severity of 0) since that's the clearest way for users of the plugin-in to communicate they are not concerned of that rule being applied to their codebase.
Overtime, it can be more work for them to apply rules they don't care about in their codebase than the rules they care about as rules become simultaneously more useful & complex.
The text was updated successfully, but these errors were encountered:
Snugug
changed the title
Enhancement: Rules perhaps should be explicitly stated to be turned off or on if not listed ESLint style
Rules explicitly turned off if not listed ESLint style
Aug 31, 2015
I originally had it so that user config overrode the default configuration instead of merging, but found that if someone only wanted to change indentation, for instance, they needed to be explicit with their whole configuration.
One option is to not have anything configured by default. Other thoughts?
It's not clear if rules not present in
sass-lint.yml
files are turned off automatically (ESLint 1.0 style) or not.With early runs of
gulp-sass-lint
, that does not seem to be the case; for codebases that have bothSass
&SCSS
litn files, it seems there's issues propagating fromgonzales-pe
unlesstrailing-semicolon
is explicitly set to0
rather than merely omitting the rule altogether:Seemingly any rule not listing in the
sass-lint.yml
should be turned off (severity of0
) since that's the clearest way for users of the plugin-in to communicate they are not concerned of that rule being applied to their codebase.Overtime, it can be more work for them to apply rules they don't care about in their codebase than the rules they care about as rules become simultaneously more useful & complex.
The text was updated successfully, but these errors were encountered: