JSHint icon indicating copy to clipboard operation
JSHint copied to clipboard

.jshintrc Support

Open whitlockjc opened this issue 11 years ago • 6 comments

JSHint supports using a file to hold your configuration. Whenever the jshint executable is invoked, you can either use the --config flag to point to the .jshintrc file or you can let jshint locate your .jshintrc file(s) based on the CWD where the executable was invoked. This is documented in the Configuration section of the JSHint Documentation. The idea would either be to fix where jshint is executed from so that jshint can work as it would from the CLI or we can update the :lt.plugins.jshint/jshint-options to allow for a workspace path or fully qualified path. Thoughts?

Typically I'd submit a pull request but I've not filed the paperwork to contribute and I'm not up to speed on ClojureScript just yet. I'll see if I can expedite these things if necessary.

whitlockjc avatar Jan 09 '14 18:01 whitlockjc

FYI, our CA is just an online form, so that's wonderfully simple and fast :)

ibdknox avatar Jan 09 '14 19:01 ibdknox

Done. Now I'll see how best to update the plugin to work. Based on your opinion, which of these solutions would you rather see? I'd much rather see jshint executed in the workspace that the file being linted since this allows jshint to do its business as if executed from the command line. Also, any pointers regarding your suggested approach would be appreciated.

whitlockjc avatar Jan 09 '14 21:01 whitlockjc

My preference would be for it to Just Work(tm) for a standard project layout where .jshintrc is in the root of the project.

doxxx avatar Jan 19 '14 16:01 doxxx

+! @doxxx

Would love to see this work, also make sure it has a go at reading "jshintConfig" from package.json as that's also an officially supported way of storing the configuration for JSHint.

mdeboer avatar Mar 26 '14 21:03 mdeboer

+1 doxxx

Armaklan avatar Nov 23 '14 08:11 Armaklan

Just released a version of this plugin. Happy to take a pull for this. No CAs are needed with LightTable anymore

cldwalker avatar Jan 14 '16 13:01 cldwalker