Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Allow configuring 'include' option as an array in sass-lint.yml #668

Closed
mischkl opened this issue May 2, 2016 · 3 comments
Closed

Allow configuring 'include' option as an array in sass-lint.yml #668

mischkl opened this issue May 2, 2016 · 3 comments
Assignees
Milestone

Comments

@mischkl
Copy link

mischkl commented May 2, 2016

We have our SCSS files in two separate folders, one for global styles and another where it is co-located with Angular components. From what I've seen in projects at two different companies this is a pretty typical file layout. In order to configure sass-lint to only see the actual paths we need, the only possibility at the moment is to specify include: '**/*.scss' and add everything else to the ignore list - but that is counter-intuitive and not really best practice since it's prone to breaking if anything in the project structure changes.

IMHO this asymmetry between include and ignore should be eliminated. include should be specifiable as an array just like ignore, thus significantly reducing the configuration complexity (because ignore can in many cases simply be ignored). ;)

@DanPurdy DanPurdy self-assigned this May 6, 2016
@DanPurdy DanPurdy added this to the 1.8.0 milestone May 6, 2016
@nmcclay
Copy link

nmcclay commented May 9, 2016

I'm +1 this, as I encountered a scenario where I would use this today.

@DanPurdy
Copy link
Member

DanPurdy commented May 9, 2016

I'm working on this as we speak, should be with you for 1.8

@mischkl
Copy link
Author

mischkl commented May 9, 2016

hooray! :) :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants