Skip to content

Commit bf67aa4

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 46c1ae9 commit bf67aa4

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
@@ -51,7 +51,8 @@ jobs:
5151
- name: Build and Run tests
5252
run: |
5353
mkdir $BISECT_DIR
54-
dune build @all @runtest --force --instrument-with bisect_ppx
54+
dune build @all @runtest
55+
dune build @runtest --force --instrument-with bisect_ppx
5556
env:
5657
BISECT_DIR: ${{ runner.temp }}/_bisect_ppx_data
5758
BISECT_FILE: ${{ runner.temp }}/_bisect_ppx_data/data

0 commit comments

Comments
 (0)