Skip to content

Commit

Permalink
build: fail on instrumentation errors
Browse files Browse the repository at this point in the history
nyc was silently failing to instrument new language features,
resulting in a failure to instrument console.js.

Refs: nodejs#20952
  • Loading branch information
Benjamin Coe committed Jun 1, 2018
1 parent 148b8ad commit 09011f0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,8 @@ coverage-build: all
"$(CURDIR)/build/jenkins/scripts/coverage/gcovr-patches-3.4.diff"); fi
if [ -d lib_ ]; then $(RM) -r lib; mv lib_ lib; fi
mv lib lib_
$(NODE) ./node_modules/.bin/nyc instrument --extension .js --extension .mjs lib_/ lib/
NODE_DEBUG=nyc $(NODE) ./node_modules/.bin/nyc instrument --extension .js \
--extension .mjs --exit-on-error lib_/ lib/
$(MAKE)

.PHONY: coverage-test
Expand Down

0 comments on commit 09011f0

Please sign in to comment.