no-invalid-position-at-import-rule needs to be disabled for scss about stylelint-config-twbs-bootstrap HOT 8 CLOSED

We don't have the bandwidth to work on cleaning the configs, but you are welcome to provide a PR assuming all the current rules are still included.

I'll make a new release for this.

from stylelint-config-twbs-bootstrap.

Hmm, perhaps we should disable the rule in the Scss config indeed. Right now the scss config inherits from the CSS config where the rule is enabled.

from stylelint-config-twbs-bootstrap.

Out of curiosity, why does the scss config inherit from css? Doesn't that defeat a part of the purpose of keeping them separate?

from stylelint-config-twbs-bootstrap.

Because many rules are common and it's been working for us for all these years.

from stylelint-config-twbs-bootstrap.

Please try this branch https://github.com/twbs/stylelint-config-twbs-bootstrap/tree/XhmikosR-patch-1

from stylelint-config-twbs-bootstrap.

The issue rn is that by inheriting from css in scss, you also inherit stylelint-config-standard. That one in turn inherits stylelint-config-recommended. That doesn't feel right.

With your current setup, your SCSS extends chain look like this:

1. stylelint-config-recommended - OK
2. stylelint-config-recommended-scss - Disables some CSS-specific stuff from the -recommended one
3. stylelint-config-recommended - This undoes step 2
4. stylelint-config-standard - OK
5. ../css - OK

I've bolded the ones you directly reference from your scss config. As you can see, step 2 is redundant. It would make sense to therefore drop the dep on that one (it does so little and is barely even maintained anymore) which would reduce confusion and "clean up" your chain so that you're extending from various CSS configs, then finally (in your SCSS one) do SCSS stuff.

from stylelint-config-twbs-bootstrap.

Will try the patch (but I'm 99% sure it'll work).

from stylelint-config-twbs-bootstrap.

It worked.

from stylelint-config-twbs-bootstrap.