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

Fix TypeError in indentation rule #524

Merged

Conversation

bgriffith
Copy link
Member

Improved the code of the indentation rule with extra checks to stop a TypeError.

While this doesn't fix the reason (Gonzales-pe's handling of CRLF), it will stop the error breaking out and allow the linter to continue.

Reported in #389, #382, #373... and lots of other reports by Windows users.

/Users/ben/workspace/github/sass-lint/lib/rules/indentation.js:57
                  'line': node.content[i + 1].start.line,

TypeError: Cannot read property 'start' of undefined

DCO 1.1 Signed-off-by: Ben Griffith gt11687@gmail.com

@DanPurdy
Copy link
Member

👍

DanPurdy added a commit that referenced this pull request Feb 12, 2016
…-issue

Fix TypeError in indentation rule
@DanPurdy DanPurdy merged commit 05f6e5b into sasstools:develop Feb 12, 2016
@bgriffith bgriffith deleted the feature/catch-crlf-indentation-issue branch April 21, 2016 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants