search

eclipsesource / jshint-eclipse

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

Trailing commas test won't run #56

Closed RobertGary1 closed 11 years ago

RobertGary1 commented 11 years ago

The code...

/jshint es5:true / var obj = { name: 'Anton', //THIS IS AN ERROR };

Should generate an error about a trailing comma. However, its not,

Running 0.9.7.20130526-111.

-Robert

ralfstx commented 11 years ago

This is almost certainly not related to the Eclipse plug-in. Please open an issue against jshint.

RobertGary1 commented 11 years ago

Unfortunately because it doesn't reproduce outside of Ecilpse I'm not able to report it against JSHint. We're mostly using IntelliJ and that exact line of code works correctly and its also using JSHint 2.1.2 :(

ralfstx commented 11 years ago

Ok, since you have reason to believe that it's not related to this version of jshint, let me check your code snippet inside and outside Eclipse.

rakis commented 11 years ago

I'm also seeing the same behavior using the latest version of the jshint-eclipse plugin. Upgraded to JSHint 2.1.7, doesn't seem to solve the issue.

ralfstx commented 11 years ago

Apparently, for JSHint 2.x you have to enable es3: true: https://github.com/jshint/jshint/issues/1131