Skip to content

Commit 9dacb34

Browse files
committed
Move opam clean into the "Show config" step
The `opam clean` must be performed before the cache is saved but it is automatically done by `setup-ocaml` so it is only really useful for Cygwin builds Move that operation because it seems that it can hide the failure of the installation of dependencies on Windows
1 parent 81faf0b commit 9dacb34

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/common.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -281,14 +281,14 @@ jobs:
281281
- name: Install Multicore Tests dependencies
282282
run: |
283283
opam install . --deps-only --with-test
284-
opam clean --all-switches --unused-repositories --logs --download-cache --repo-cache
285284
286-
- name: Show configuration
285+
- name: Show configuration and clean up
287286
run: |
288287
opam exec -- ocamlc -config
289288
opam config list
290289
opam exec -- dune printenv
291290
opam list --columns=name,installed-version,repository,synopsis-or-target
291+
opam clean --all-switches --unused-repositories --logs --download-cache --repo-cache
292292
293293
- name: Save OPAM state (Cygwin)
294294
uses: actions/cache/save@v3

0 commit comments

Comments
 (0)