fmarier/node-libravatar — pull request #3 by chilts (2011-12-30 09:14:24+00:00)
Title: No need for jsonlint since by using 'npm test' it reads package.json which means it has to be valid

Ok, so I've just gone round in a circle and removed jsonlint completely.

Go to your command line and instead of typing 'make test', just type 'npm test'. What this does is read the package.json file, extract the command 'test' from the 'scripts' section and runs it.

Of course this means that package.json _must_ be valid.

Also, if you want to know why ./node_modules/.bin/tap is now just 'tap' is because npm adds the ./node_modules/.bin/ path to any command in the scripts section of package.json automatically.

Again, very much an optional patch and might help you without having to suppress the jsonlint output. Also, you can always add the 'test' section of the Makefile back in to just be 'npm test'. :)