twbs/stylelint-config-twbs-bootstrap

`selector-class-pattern` change in >= v3.0.0-alpha1

danny007in opened this issue · 6 comments

Expected keyframe name to be kebab-case keyframes-name-pattern

@keyframes flipInX {

Expected class selector to be kebab-case selector-class-pattern

.svg-inline--fa,

That intended. Feel free to disable the rule on your project.

Actually, let's keep the issue open. I will rethink about it, but keep in mind this is a major version.

Either way the change was intentional, but if if people prefer the old way I guess we could revisit this change.

@ffoodd do you think we should revert the selector-class-pattern change to our custom rule which was less restrictive? Now, it's strictly kebab case.

I'd be in favor of keeping it, as we only use kebab-case. Linter rules are meant to reinforce our coding conventions, so it totally makes sense this way.

All right, I agree, but do note that we too had to override this in one case in our docs.

I'll release v3.0.0 tomorrow :)

Closing since we are going to stick with the new behavior for the time being.