Skip to content
Permalink
Browse files

tools: exclude benchmark code from coverage report

Refs: nodejs/build#1676 (comment)

PR-URL: #25841
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ben Coe <bencoe@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
  • Loading branch information...
Trott authored and addaleax committed Jan 31, 2019
1 parent 7cb9439 commit f395728b325d2acb9efd343647dcbd8d60d3cfa9
Showing with 1 addition and 1 deletion.
  1. +1 −1 Makefile
@@ -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)

0 comments on commit f395728

Please sign in to comment.
You can’t perform that action at this time.