search

sasstools / sass-lint

Pure Node.js Sass linting
MIT License
1.77k stars 532 forks source link

class-name-format ignore: wildcard possible? #750

Open isGabe opened 8 years ago

isGabe commented 8 years ago

Using 1.8.0

I'm wondering if any sort of wildcard is possible for the list of ignored selectors in the class-name-format. In our project we have a ton of selectors related to the Field Manager plugin that all start with .fm-* and it would so great to ignore all of them without having to list them all out, as there are dozens, and there will no doubt be dozens more.

DanPurdy commented 8 years ago

There isn't this option and i'm also not sure it's something I would like to see added either as it opens up a lot of edge cases. I think once the ability to disable linters via comments is enabled it would be better to use these to disable this rule instead of trying to apply this to all name format rules. Happy to discuss though.

isGabe commented 8 years ago

it opens up a lot of edge cases.

Hadn't considered that, but if that's the case I can understand the hesitation.

I think once the ability to disable linters via comments is enabled it would be better to use these to disable this rule instead of trying to apply this to all name format rules.

Yeah this is one of the few (maybe only) things still in the way of us moving over from scss-lint for our main project. We have quite a few places we're using the comments for selectors that target third party stuff that we can't really change to fit our coding conventions.

Seems like disabling via comments would probably be an OK way to deal with it, as well.

Interestingly the use case is selectors like .fm-featured_video-wrapper which are triggering the class-name-format linter in sass-lint, but not SelectorFormat in scss-lint, and they are set to hyphenatedbem and hyphenated_BEM respectively.