add jshint devDependency and a make target for jshint

This commit is contained in:
Philipp Borgers 2013-01-21 11:46:29 +01:00
parent a24102c237
commit fc7b4fe550
2 changed files with 9 additions and 2 deletions

View File

@ -6,7 +6,8 @@ params := $(connectionString)
node-command := xargs -n 1 -I file node file $(params) node-command := xargs -n 1 -I file node file $(params)
.PHONY : test test-connection test-integration bench test-native build/default/binding.node .PHONY : test test-connection test-integration bench test-native \
build/default/binding.node jshint
help: help:
@echo "make prepare-test-db [connectionString=pg://<your connection string>]" @echo "make prepare-test-db [connectionString=pg://<your connection string>]"
@ -50,3 +51,6 @@ prepare-test-db:
@echo "***Preparing the database for tests***" @echo "***Preparing the database for tests***"
@find script/create-test-tables.js | $(node-command) @find script/create-test-tables.js | $(node-command)
jshint:
@echo "***Starting jshint***"
@./node_modules/.bin/jshint lib

View File

@ -12,6 +12,9 @@
"dependencies" : { "dependencies" : {
"generic-pool" : "2.0.2" "generic-pool" : "2.0.2"
}, },
"devDependencies" : {
"jshint" : "*"
},
"scripts" : { "scripts" : {
"test" : "make test-all connectionString=pg://postgres@localhost:5432/postgres", "test" : "make test-all connectionString=pg://postgres@localhost:5432/postgres",
"prepublish": "rm -r build || (exit 0)", "prepublish": "rm -r build || (exit 0)",