Skip to content

Commit 7959b6d

Browse files
richardlaujasnell
authored andcommitted
build: update coverage Makefile target comments
Update the comments for the `coverage` Makefile target. - Source files under `lib` are no longer instrumented since d1dee49 - Fix the name of the related CI job. PR-URL: #39365 Refs: #25157 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 74e319f commit 7959b6d

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

Makefile

+4-5
Original file line numberDiff line numberDiff line change
@@ -210,11 +210,10 @@ coverage-clean:
210210
-type f -exec $(RM) {} \;
211211

212212
.PHONY: coverage
213-
# Build and test with code coverage reporting. Leave the lib directory
214-
# instrumented for any additional runs the user may want to make.
215-
# For C++ coverage reporting, this needs to be run in conjunction with configure
216-
# --coverage. html coverage reports will be created under coverage/
217-
# Related CI job: node-test-commit-linux-coverage
213+
# Build and test with code coverage reporting. HTML coverage reports will be
214+
# created under coverage/. For C++ coverage reporting, this needs to be run
215+
# in conjunction with configure --coverage.
216+
# Related CI job: node-test-commit-linux-coverage-daily
218217
coverage: coverage-test ## Run the tests and generate a coverage report.
219218

220219
.PHONY: coverage-build

0 commit comments

Comments
 (0)