Skip to content


Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP


Makefile enhancements #2

wants to merge 2 commits into from

3 participants


Exposes a "make check" (per GNU standards) and becomes tolerant to presence or absence of a local mocha.

tj commented

IMO mocha should always be localized like any other dep


@visionmedia if it is localized it will be used as such by make, if it isn't it will attempt to work anyway.
b239fb4 just expands PATH to prepend node_modules/.bin, in the same way done by "npm test"

@donpark donpark referenced this pull request

Feature: chaining #14


Closing. You will get mocha localized after npm install, which you need to run anyways to pick up the other dependencies that are required for development.

@gjohnson gjohnson closed this

What about the check alias for test ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +3 −3 Makefile
6 Makefile
@@ -1,7 +1,7 @@
- @./node_modules/.bin/mocha \
+check test:
+ PATH=./node_modules/.bin:${PATH} mocha \
--require should \
--reporter spec
-.PHONY: test
+.PHONY: test
Something went wrong with that request. Please try again.