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

Empty Scss file breaks linting #91

Closed
elidupuis opened this issue Sep 3, 2015 · 0 comments
Closed

Empty Scss file breaks linting #91

elidupuis opened this issue Sep 3, 2015 · 0 comments
Assignees
Milestone

Comments

@elidupuis
Copy link

I'm running 1.1.0 CLI and getting the following error:

/Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/lib/rules/extends-before-mixins.js:11
    ast.traverseByType('block', function (block) {
        ^
TypeError: undefined is not a function
    at Object.module.exports.detect (/Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/lib/rules/extends-before-mixins.js:11:9)
    at /Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/index.js:30:25
    at Array.forEach (native)
    at Function.sassLint.lintText (/Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/index.js:29:9)
    at /Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/index.js:73:21
    at Array.forEach (native)
    at Function.sassLint.lintFiles (/Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/index.js:72:9)
    at detectPattern (/Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/bin/sass-lint.js:16:18)
    at /Users/eli/.nvm/v0.12.7/lib/node_modules/sass-lint/bin/sass-lint.js:58:5
    at Array.forEach (native)

I was able to track this down to an empty .scss file. After deleting the empty file, everything runs fine!

@DanPurdy DanPurdy added the bug label Sep 3, 2015
@Snugug Snugug added this to the 1.2.0 milestone Sep 6, 2015
@Snugug Snugug self-assigned this Sep 6, 2015
Snugug added a commit to Snugug/sass-lint that referenced this issue Sep 6, 2015
Only run parsing for files with stuff in them

Resolves sasstools#91
donabrams pushed a commit to donabrams/sass-lint that referenced this issue Nov 19, 2015
Only run parsing for files with stuff in them

Resolves sasstools#91
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