Added jshint-eclipse to available editor integration #133

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

pulse00 commented Oct 9, 2012

No description provided.

Contributor

brentlintner commented Oct 9, 2012

Awesome. Will merge this in, although it seems the absence of a newline after line 26 causes the MD to not be formatted correctly (in github). Mind adding that quickly? I can also do that as well, if not. Thanks!

Contributor

pulse00 commented Oct 9, 2012

should be ok now.

Contributor

brentlintner commented Oct 9, 2012

Ok, cool. I just merged this to master, although I noticed there was a - in that newline fix which was still causing a formatting issue, so (instead of going back and forth once more) I went ahead fixed it up myself (hope that is alright). :-) Thanks again! Closing.

Contributor

pulse00 commented Oct 9, 2012

thanks a lot and sorry for the newline troubles ;)

Contributor

brentlintner commented Oct 9, 2012

No worries! :-)

@chadaustin chadaustin pushed a commit to imvu/node-jshint that referenced this pull request Mar 13, 2013

@goatslacker @valueof goatslacker + valueof Accumulate all undef errors to be processed at the very end.
This approach makes sure that errors caused by undef are not in
the way of other options such as latedef. The code pushes all
warning about undefined variables into an array and at the
end of processing checks if any of those variables were defined
later in the code and can be legally used due to hoisting.

Fixes GH-133.
Fixes GH-347.

P.S. The only thing is that it can mess up the maxerr option but
we need to refactor it anyway.
ce7626a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment