Commit 1630a56
committed
tools: avoid building twice in coverage jobs
PR-URL: #61899
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent dba9001 commit 1630a56
2 files changed
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -71,11 +71,11 @@ jobs: | |||
| 71 | 71 | run: npx envinfo | |
| 72 | 72 | - name: Install gcovr | |
| 73 | 73 | run: pip install gcovr==7.2 | |
| 74 | - - name: Build | ||
| 75 | - run: make build-ci -j4 V=1 CONFIG_FLAGS="--error-on-warn --coverage --without-intl" | ||
| 74 | + - name: Configure | ||
| 75 | + run: ./configure --verbose --error-on-warn --coverage --without-intl | ||
| 76 | 76 | # TODO(bcoe): fix the couple tests that fail with the inspector enabled. | |
| 77 | 77 | # The cause is most likely coverage's use of the inspector. | |
| 78 | - - name: Test | ||
| 78 | + - name: Build and test | ||
| 79 | 79 | run: NODE_V8_COVERAGE=coverage/tmp make test-cov -j4 V=1 TEST_CI_ARGS="-p dots --measure-flakiness 9" || exit 0 | |
| 80 | 80 | - name: Report JS | |
| 81 | 81 | run: npx c8 report --check-coverage | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -71,11 +71,11 @@ jobs: | |||
| 71 | 71 | run: npx envinfo | |
| 72 | 72 | - name: Install gcovr | |
| 73 | 73 | run: pip install gcovr==7.2 | |
| 74 | - - name: Build | ||
| 75 | - run: make build-ci -j4 V=1 CONFIG_FLAGS="--error-on-warn --coverage" | ||
| 74 | + - name: Configure | ||
| 75 | + run: ./configure --verbose --error-on-warn --coverage | ||
| 76 | 76 | # TODO(bcoe): fix the couple tests that fail with the inspector enabled. | |
| 77 | 77 | # The cause is most likely coverage's use of the inspector. | |
| 78 | - - name: Test | ||
| 78 | + - name: Build and test | ||
| 79 | 79 | run: NODE_V8_COVERAGE=coverage/tmp make test-cov -j4 V=1 TEST_CI_ARGS="-p dots --measure-flakiness 9" || exit 0 | |
| 80 | 80 | - name: Report JS | |
| 81 | 81 | run: npx c8 report --check-coverage | |
0 commit comments