Skip to content

Commit

Permalink
tools: exclude benchmark code from coverage report
Browse files Browse the repository at this point in the history
Refs: nodejs/build#1676 (comment)

PR-URL: nodejs#25841
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Ben Coe <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
  • Loading branch information
Trott committed Feb 2, 2019
1 parent 988482e commit 8198ca0
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 @@ -242,7 +242,7 @@ coverage-test: coverage-build

COV_REPORT_OPTIONS = --reporter=html \
--temp-directory=out/$(BUILDTYPE)/.coverage --omit-relative=false \
--resolve=./lib --exclude="deps/" --exclude="test/" --exclude="tools/" \
--resolve=./lib --exclude="benchmark/" --exclude="deps/" --exclude="test/" --exclude="tools/" \
--wrapper-length=0
ifdef COV_ENFORCE_THRESHOLD
COV_REPORT_OPTIONS += --check-coverage --lines=$(COV_ENFORCE_THRESHOLD)
Expand Down

0 comments on commit 8198ca0

Please sign in to comment.