Skip to content
This repository has been archived by the owner on Mar 23, 2022. It is now read-only.

Please update the plugin's embedded version of JSLint.js #64

Open
gmazza opened this issue Jun 23, 2013 · 2 comments
Open

Please update the plugin's embedded version of JSLint.js #64

gmazza opened this issue Jun 23, 2013 · 2 comments

Comments

@gmazza
Copy link

gmazza commented Jun 23, 2013

Hi the latest JSLint.js has many more settings (such as the "vars" setting):
https://github.com/douglascrockford/JSLint/blob/master/jslint.js#L288

than the three-year old one packaged with the plugin:
https://github.com/davidB/yuicompressor-maven-plugin/blob/master/src/main/resources/jslint.js#L53

Right now I can't use /jslint vars: true/ in a JavaScript file because the yuicompressor-maven-plugin's jslint.js version doesn't have that option (the plugin's jslint goal just returns an error and halts.)

Could you please update the plugin to the latest JSLint.js?

@davidB
Copy link
Owner

davidB commented Jun 28, 2013

In an other issue #63 you wrote that you use jslint-maven-plugin. So what is the issue ?

After (re) reading both issue, may be I should removed jslint from the next version ?

@gmazza
Copy link
Author

gmazza commented Jun 28, 2013

This is dependent on #63. If you decide to keep it, you should update it to the latest one as it saves people a lot of time and aggravation as they can't understand why their jslint settings aren't working.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants