Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Makefile enhancements #2

Closed
wants to merge 2 commits into from

3 participants

@strk

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

@tj
Owner
tj commented

IMO mocha should always be localized like any other dep

@strk

@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
Closed

Feature: chaining #14

@gjohnson
Owner

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
@strk

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
View
6 Makefile
@@ -1,7 +1,7 @@
-test:
- @./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.