Merge pull request #285 from weaveworks/test-bundle

Fail make static on build JS build errors

Makefile

@@ -1,4 +1,4 @@
-.PHONY: all deps static clean client-lint client-sync
+.PHONY: all deps static clean client-lint client-test client-sync
 
 # If you can use Docker without being root, you can `make SUDO= <target>`
 SUDO=sudo

circle.yml

@@ -35,10 +35,10 @@ dependencies:
 test:
   override:
     - cd $SRCDIR; ./bin/lint .
+    - cd $SRCDIR; make client-test
     - cd $SRCDIR; make static
     - cd $SRCDIR; make
     - cd $SRCDIR; ./bin/test -slow
-    - cd $SRCDIR; make client-test
     - cd $SRCDIR/experimental; make
   post:
     - goveralls -repotoken $COVERALLS_REPO_TOKEN -coverprofile=$SRCDIR/profile.cov -service=circleci || true

client/webpack.config.js

@@ -26,6 +26,7 @@ var config = {
     sourcePrefix: '  '
   },
 
+  bail: !DEBUG, // fail on first error when building release
   cache: DEBUG,
   debug: DEBUG,
   devtool: DEBUG ? '#inline-source-map' : false,