Problem

The rule in https://github.com/xojs/stylelint-config-xo-scss/blob/master/index.js#L21 requires an extension for all imports. But the next line prohibites to use the .scss extension.

If I want to import e.g some color values from an seperated color.scss to an other scss file, then I want to use it like that:

Minimal Reproducible Example

@import './color.scss'

body {
    background: $color-silver;
}

But the config does not allow this.
If I remove the scss extension at the end, the rule scss/at-import-partial-extension throws an eror.
If I add the scss extension the scss/at-import-partial-extension-blacklist rule throws an error.
So it's not possible to solve the errors at this point.

Possible Solution

IMO it would be more consistent if we delete the scss/at-import-partial-extension-blacklist. Then just every import needs the extension.

OR

We modify the scss/at-import-partial-extension to something like that (needs extra PR in stylelint-scss):

'scss/at-import-partial-extension': ['always', , { "except": ["scss"] }]

About

Since version 0.10.0, the following error causes when extending this config:

Invalid Option: Invalid option value "true" for rule "scss/at-import-partial-extension"

Minimal Reproducible Example

$ mkdir stylelint-config-xo-scss-bug && cd $_
$ yarn add stylelint [email protected]
$ echo 'module.exports = {extends: "stylelint-config-xo-scss"}' > stylelint.config.js
$ echo '/* for testing */' > test.scss
$ yarn stylelint test.scss
yarn run v1.19.2
warning package.json: No license field
/.../stylelint-config-xo-scss-bug/node_modules/.bin/stylelint test.scss

Invalid Option: Invalid option value "true" for rule "scss/at-import-partial-extension"

error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Work Around

Add this line into your stylelint.config.js:

  rules: {'scss/at-import-partial-extension': 'always'}

Possible Solution

This bug is introduced into 86e7d26.

We can easily fix this by modifying option value into "always".
However I wonder why the test miss this bug, and I guess the test is broken. It should be fixed.

Thank you 😄