Skip to content

Commit

Permalink
Linting with JSHint: run npm install --dev for dependency
Browse files Browse the repository at this point in the history
  • Loading branch information
Tom MacWright committed May 10, 2011
1 parent 2929ba2 commit d71ab50
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 3 deletions.
5 changes: 4 additions & 1 deletion Makefile
Expand Up @@ -8,6 +8,9 @@ ext:
wget --no-check-certificate https://github.com/developmentseed/openlayers_slim/raw/v1.9/OpenLayers.js -O ext/OpenLayers.js
wget --no-check-certificate https://github.com/stamen/modestmaps-js/raw/c202b0a274f4235c4ebd8362fff11d88a2c0fc8f/modestmaps.min.js -O ext/modestmaps.js

lint:
./node_modules/.bin/jshint control/lib/*.js control/*/*.js

build/wax.ol.min.js:
cat lib/*.js control/lib/*.js control/ol/*.js > build/wax.ol.js
uglifyjs build/wax.ol.js > build/wax.ol.min.js
Expand All @@ -23,7 +26,7 @@ build/wax.mm.min.js:
build_setup:
mkdir build

build: build_setup build/wax.ol.min.js build/wax.g.min.js build/wax.mm.min.js
build: build_setup build/wax.ol.min.js build/wax.g.min.js build/wax.mm.min.js lint

clean:
rm -rf build
Expand Down
2 changes: 1 addition & 1 deletion control/lib/tooltip.js
Expand Up @@ -75,5 +75,5 @@ wax.tooltip.unselect = function(feature, context, layer_id, evt) {
.removeClass('hidden')
.show();

$(context).triggerHandler('removedtooltip', [feature, context, evt])
$(context).triggerHandler('removedtooltip', [feature, context, evt]);
};
9 changes: 8 additions & 1 deletion package.json
@@ -1,6 +1,13 @@
{
"author": "Development Seed <info@developmentseed.org>",
"name": "wax",
"author": "Tom MacWright <macwright@gmail.com>",
"version": "1.2.2",
"files": [ "build" ]
"files": [
"build"
],
"devDependencies": {
"jshint": "0.2.x",
"docco": "0.3.x"
}
}

0 comments on commit d71ab50

Please sign in to comment.