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 the error range construction and line reporting for line zero errors #65

Merged
merged 2 commits into from
May 30, 2014

Conversation

park9140
Copy link

Resolves #35, also fixes implementation of showing all line lint errors.

@@ -119,7 +119,7 @@ class Linter
level = 'warning'
else
level = @defaultLevel

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌋

@iam4x
Copy link
Contributor

iam4x commented May 30, 2014

I'm good with it ;)

iam4x pushed a commit that referenced this pull request May 30, 2014
…or-line-zero

Fix the error range construction and line reporting for line zero errors
@iam4x iam4x merged commit 4140f01 into master May 30, 2014
@iam4x iam4x added the bug label May 30, 2014
@iam4x iam4x added this to the 0.4.9 milestone May 30, 2014
@norcalli
Copy link

This is broken again for csslint, which ends up setting rowStart and rowEnd to NaN because of a full file warning. I submitted a possible fix as #103 (the other way is to catch isNaN(rowStart) along side rowStart == -1, but I figured catching it earlier was better than anything.

@steelbrain steelbrain deleted the fix-error-range-construction-for-line-zero branch July 27, 2015 03:34
@park9140 park9140 removed their assignment Mar 19, 2016
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.

Linter Causing error every time it lints
3 participants