Skip to content

Commit 34cc510

Browse files
committed
Run the tests first and instrumented after that
Pragmatically when running locally I have observed that this sequence usually ends up resolving the partial coverage report that otherwise gets generated sometimes. That is a an experimental change to adopt dune `3.23.1` to work around the partial coverage issue. Better fix left as future work.
1 parent 0285f7f commit 34cc510

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,8 @@ jobs:
5555
- name: Build and Run tests
5656
run: |
5757
mkdir $BISECT_DIR
58-
dune build @all @runtest --force --instrument-with bisect_ppx
58+
dune build @all @runtest
59+
dune build @runtest --force --instrument-with bisect_ppx
5960
env:
6061
BISECT_DIR: ${{ runner.temp }}/_bisect_ppx_data
6162
BISECT_FILE: ${{ runner.temp }}/_bisect_ppx_data/data

0 commit comments

Comments
 (0)