Skip to content

Commit 562f59e

Browse files
authored
CI: Run tests without coverage in merge checks (#3917)
## Why Reduce waiting for merge queue.
1 parent 837bbcf commit 562f59e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/push.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,17 +86,17 @@ jobs:
8686
pip3 install wheel==0.45.1
8787
8888
- name: Run tests without coverage
89-
# We run tests without coverage on PR because we don't make use of coverage information
89+
# We run tests without coverage on PR, merge_group, and schedule because we don't make use of coverage information
9090
# and would like to run the tests as fast as possible. We run it on schedule as well, because that is what
9191
# populates the cache and cache may include test results.
92-
if: ${{ github.event_name == 'pull_request' || github.event_name == 'schedule' }}
92+
if: ${{ github.event_name == 'pull_request' || github.event_name == 'merge_group' || github.event_name == 'schedule' }}
9393
env:
9494
ENVFILTER: DATABRICKS_BUNDLE_ENGINE=${{ matrix.deployment }}
9595
run: make test
9696

9797
- name: Run tests with coverage
98-
# Still run 'make cover' on push to main and merge checks to make sure it does not get broken.
99-
if: ${{ github.event_name != 'pull_request' && github.event_name != 'schedule' }}
98+
# Only run 'make cover' on push to main to make sure it does not get broken.
99+
if: ${{ github.event_name == 'push' }}
100100
env:
101101
ENVFILTER: DATABRICKS_BUNDLE_ENGINE=${{ matrix.deployment }}
102102
run: make cover

0 commit comments

Comments
 (0)