From 5bafcd42f6a6343a50f6365233c09134173e85bb Mon Sep 17 00:00:00 2001 From: ehg Date: Mon, 23 Nov 2015 13:32:23 +0000 Subject: [PATCH] Build: Install pre-commit githook on `make run` 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` --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index dcc53fcc5be84..a3cdf15c27ed0 100644 --- a/Makefile +++ b/Makefile @@ -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,