Commit 6e06b2f
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 d2c0da2 commit 6e06b2f
1 file changed
Lines changed: 4 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -209,11 +209,10 @@ coverage-clean: | |||
| 209 | 209 | -type f -exec $(RM) {} \; | |
| 210 | 210 | ||
| 211 | 211 | .PHONY: coverage | |
| 212 | - # Build and test with code coverage reporting. Leave the lib directory | ||
| 213 | - # instrumented for any additional runs the user may want to make. | ||
| 214 | - # For C++ coverage reporting, this needs to be run in conjunction with configure | ||
| 215 | - # --coverage. html coverage reports will be created under coverage/ | ||
| 216 | - # Related CI job: node-test-commit-linux-coverage | ||
| 212 | + # Build and test with code coverage reporting. HTML coverage reports will be | ||
| 213 | + # created under coverage/. For C++ coverage reporting, this needs to be run | ||
| 214 | + # in conjunction with configure --coverage. | ||
| 215 | + # Related CI job: node-test-commit-linux-coverage-daily | ||
| 217 | 216 | coverage: coverage-test ## Run the tests and generate a coverage report. | |
| 218 | 217 | ||
| 219 | 218 | .PHONY: coverage-build | |
0 commit comments