Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added "include" as allowed at-rule in hasOnlyAllowedAtRules function #146

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DKvistgaard
Copy link

This extends the fix in #140, but fixes linting for selectors that only includes at-rules with @include within it for better Sass/SCSS support.

The error:

/** @define foo */

.foo {}

// This selector is ignored even though it's an error, as it only contains
// an @include at-rule
.bar {
    @include col-init();
}

// This is also ignored even though it contains the allow @extend at-rule as it
// also includes the @include at-rule and no other styling properties
.test {
    @include col-init();

    @extend %test-extend;
}

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant