Skip to content

Commit e9c47f2

Browse files
committed
Use conf-timeout opam package to ensure coreutils is installed when testing
This change was recommended when Beluga v1.1.2 was released: ocaml/opam-repository#26266
1 parent 36e541d commit e9c47f2

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

.github/workflows/build-and-test.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,15 +72,15 @@ jobs:
7272
ocaml-compiler: ${{ matrix.ocaml-compiler }}
7373
dune-cache: true
7474

75+
- name: Install coreutils on MacOS
76+
if: runner.os == 'macOS'
77+
run: brew install coreutils
78+
7579
- name: Install dependencies
7680
run: opam install --locked --deps-only --with-test --with-doc ./beluga.opam
7781

7882
- name: Build
7983
run: opam exec -- make
8084

81-
- name: Install coreutils on MacOS
82-
if: runner.os == 'macOS'
83-
run: brew install coreutils
84-
8585
- name: Test
8686
run: opam exec -- make test

beluga.opam

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ depends: [
2424
"sedlex" {>= "2.5"}
2525
"omd" {>= "1.3.2"}
2626
"uri" {>= "4.2.0"}
27+
"conf-timeout" {with-test}
2728
"ocamlformat" {= "0.25.1" & with-test}
2829
"yojson" {>= "2.0.2" & with-test}
2930
"ounit2" {>= "2.2.6" & with-test}

dune-project

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
(>= 1.3.2))
4545
(uri
4646
(>= 4.2.0))
47+
(conf-timeout :with-test)
4748
(ocamlformat
4849
(and
4950
(= 0.25.1)

0 commit comments

Comments
 (0)