Skip to content

Commit

Permalink
Global node-jshint
Browse files Browse the repository at this point in the history
Removes local node_module to once again rely on the globally installed version of jshint via npm. There are too many issues with permissions when using a local version.

Closes #18 and #19.
  • Loading branch information
Jamie Hoover ⚔ committed Apr 7, 2012
1 parent d6f3537 commit 1082d97
Show file tree
Hide file tree
Showing 42 changed files with 10 additions and 7,009 deletions.
4 changes: 2 additions & 2 deletions JSHint.sublime-build
@@ -1,9 +1,9 @@
{ {
"selector": "source.js", "selector": "source.js",


"path": "/usr/local/bin:${packages}/JSHint/node_modules/jshint/bin", "path": "/usr/local/bin",


"cmd": ["${packages}/JSHint/node_modules/jshint/bin/hint", "$file", "--reporter", "${packages}/JSHint/reporter.js"], "cmd": ["jshint", "$file", "--reporter", "${packages}/JSHint/reporter.js"],


"file_regex": "file:\\s(.+)\\]", "file_regex": "file:\\s(.+)\\]",
"line_regex": "(\\d+),(\\d+)", "line_regex": "(\\d+),(\\d+)",
Expand Down
8 changes: 6 additions & 2 deletions README.md
Expand Up @@ -2,13 +2,17 @@


The best JavaScript syntax checker: [JSHint](http://www.jshint.com/) for the best text editor: [Sublime Text 2](http://www.sublimetext.com/2) The best JavaScript syntax checker: [JSHint](http://www.jshint.com/) for the best text editor: [Sublime Text 2](http://www.sublimetext.com/2)


**Prerequisites:** node 0.6.x and [Sublime Package Control](http://wbond.net/sublime_packages/package_control/installation) **Prerequisites:** [node-jshint](https://github.com/jshint/node-jshint) and [Sublime Package Control](http://wbond.net/sublime_packages/package_control/installation)


**Mac OS X:** Install node with homebrew `brew update` `brew install node`. Make sure you have either the latest Command Line Tools or Xcode before installing homebrew. **Mac OS X:** Install node with homebrew `brew update` `brew install node`. Make sure you have either the latest Command Line Tools or Xcode before installing homebrew.


**Windows:** Node should be in your `CLASSPATH` in environment variables, `C:\Program Files (x86)\nodejs` **Windows:** Node should be in your `CLASSPATH` in environment variables, `C:\Program Files (x86)\nodejs`


##Install with Package Control ##Install node-jshint with npm

npm install -g jshint

##Install JSHint with Package Control in Sublime Text 2


1. `command`-`shift`-`P` *or `control`-`shift`-`P` in Linux/Windows* 1. `command`-`shift`-`P` *or `control`-`shift`-`P` in Linux/Windows*
2. type `install p`, select `Package Control: Install Package` 2. type `install p`, select `Package Control: Install Package`
Expand Down
1 change: 0 additions & 1 deletion node_modules/.bin/jshint

This file was deleted.

3 changes: 0 additions & 3 deletions node_modules/jshint/.gitmodules

This file was deleted.

3 changes: 0 additions & 3 deletions node_modules/jshint/.jshintignore

This file was deleted.

40 changes: 0 additions & 40 deletions node_modules/jshint/.jshintrc

This file was deleted.

3 changes: 0 additions & 3 deletions node_modules/jshint/.npmignore

This file was deleted.

9 changes: 0 additions & 9 deletions node_modules/jshint/HELP

This file was deleted.

4 changes: 0 additions & 4 deletions node_modules/jshint/LICENSE

This file was deleted.

71 changes: 0 additions & 71 deletions node_modules/jshint/README.md

This file was deleted.

2 changes: 0 additions & 2 deletions node_modules/jshint/bin/hint

This file was deleted.

140 changes: 0 additions & 140 deletions node_modules/jshint/lib/cli.js

This file was deleted.

0 comments on commit 1082d97

Please sign in to comment.