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

Ignored file still being processed #129

Closed
mven opened this issue Sep 7, 2015 · 7 comments
Closed

Ignored file still being processed #129

mven opened this issue Sep 7, 2015 · 7 comments
Milestone

Comments

@mven
Copy link
Contributor

mven commented Sep 7, 2015

sass-lint -v -i tests/sass/border-zero.scss tests/sass/*.scss

This results in border-zero.scss being processed still.

@DanPurdy
Copy link
Member

I will look into this tomorrow hopefully, I've been updating the CLI functionality and this is the next step there.. Will update here when I know, thanks for the report!

@DanPurdy
Copy link
Member

Ignore the above, I swear someone deleted their comment!

@wingleung
Copy link

I deleted my comment because I thought you were addressing mven's issue. My issue turned out to be another bug (Maybe) that had nothing to do with cli.

I'm using gulp-sass-lint. If I define some files to ignore in .sass-lint.yml. It doesn't get picked up. Instead I had to ignore the files in the gulp.src method gulp.src(['sass/**/*', '!sass/vendor/**/*'])

Sorry for the misunderstanding.

@DanPurdy
Copy link
Member

Ah that makes sense, no problem!

@mik01aj
Copy link

mik01aj commented Oct 15, 2015

I can confirm the bug with sass-lint 1.3.0.

Running sass-lint -i 'mockups/**/vendor/**/*' 'mockups/**/*.scss' fails with:

Error: Parsing error at mockups/src/sass/vendor/bootstrap/_glyphicons.scss: Please check validity of the block starting from line #14

And this file shouldn't be ever touched.

@bgriffith bgriffith added the cli label Oct 17, 2015
@DanPurdy
Copy link
Member

Looking into this now.

@DanPurdy DanPurdy added bug cli and removed cli labels Oct 18, 2015
@DanPurdy
Copy link
Member

PR incoming to fix this

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

5 participants