Skip to content

Commit

Permalink
Build: Install pre-commit githook on make run
Browse files Browse the repository at this point in the history
The rationale for this is that we'd like more exposure for the GPLv2
contribution notice. Installing them on `make run` seems like a good way
to do this.

The way that `bin/pre-commit` is set up now, this
will also force the running of i18nlint and eslint checks. Which is a
Good Thing in any case.

Of course, these checks can be bypassed using `git commit --no-verify`
  • Loading branch information
ehg committed Nov 23, 2015
1 parent 01b1201 commit 5bafcd4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ export NODE_PATH := server:shared:$(THIS_DIR)
install: node_modules

# Simply running `make run` will spawn the Node.js server instance.
run: install build
run: githooks-commit install build
@$(NODE) build/bundle-$(CALYPSO_ENV).js

# a helper rule to ensure that a specific module is installed,
Expand Down

0 comments on commit 5bafcd4

Please sign in to comment.