search

eclipsesource / jshint-eclipse

JSHint integration for the Eclipse IDE
http://github.eclipsesource.com/jshint-eclipse/
146 stars 31 forks source link

JSHINT 2.3.0 is not a valid JSHint library #75

Closed dertin closed 10 years ago

dertin commented 10 years ago

I tried, but does not pass the test. says that is out of range index in the tests.

 checkWithNullCode(com.eclipsesource.jshint.JSHint_Test): out of range index
 checkWithNullText(com.eclipsesource.jshint.JSHint_Test): out of range index
 visualToCharIndex(com.eclipsesource.jshint.JSHint_Test): out of range index
 createProblem_cutsOffTrailingPeriodFromMessage(com.eclipsesource.jshint.JSHint_Test): out of range index
 checkWithRhinoException(com.eclipsesource.jshint.JSHint_Test): out of range index
 loadCustomWithGarbage(com.eclipsesource.jshint.JSHint_Test): out of range index
 configureWithNull(com.eclipsesource.jshint.JSHint_Test): out of range index
 createProblem_mapsVisualToRealIndex(com.eclipsesource.jshint.JSHint_Test): out of range index
 noErrorsWithEmptyConfig(com.eclipsesource.jshint.JSHint_Test): out of range index
 checkWithOnlyWhitespace(com.eclipsesource.jshint.JSHint_Test): out of range index
ddasilva commented 10 years ago

Same issue here. Would really like to upgrade to latest JSHint to use the ignore:start / ignore:end directives.

ralfstx commented 10 years ago

Filed https://github.com/jshint/jshint/issues/1422

ralfstx commented 10 years ago

Seems that this depends on https://github.com/jshint/jshint/issues/1333

tdelmas commented 10 years ago

As rhino seams dead since one year, is it possible to add an option to run jshint in interpretive mode ? It is slower but it give the choice betwenn be stuck in an older jshint version and the new one.

paulvi commented 10 years ago

is it possible to add an option to run jshint in interpretive mode ? @tdelmas means run jshint with rhino jshint/jshint#1333

In the long run (1-2 year ahead) I think this plugin should be able to run using Java 8 Nashorn as well. See #83

paulvi commented 10 years ago

https://github.com/jshint/jshint/issues/1333 is solved

ralfstx commented 10 years ago

Fixed by update to jshint 2.4.3 #90

SamHasler commented 10 years ago

Is it possible to make a release with this fix?