ci: fix caching and other improvement

.github/workflows/main.yml

@@ -1,35 +1,76 @@
-name: build
+name: Build and test
-on: [push, pull_request]
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
 jobs:
   run:
     name: Build
     strategy:
+      fail-fast: false
       matrix:
-        #os: [macos-latest, ubuntu-latest, windows-latest]
+        os:
-        os: [macos-latest, ubuntu-latest]
+            - macos-latest
-        ocaml: [ '4.08.0', '4.03.0', '4.11.0' ]
+            - ubuntu-latest
+            - windows-latest
+        ocaml:
+            - '4.08.1'
+            - '4.03.0'
+            - '4.12.0'
+
     runs-on: ${{ matrix.os }}
     steps:
-    - uses: actions/checkout@main
+      - uses: haya14busa/action-cond@v1
-    - uses: avsm/setup-ocaml@v1
+        id: path
         with:
-        ocaml-version: ${{ matrix.ocaml }}
+          cond: ${{ matrix.os == 'windows-latest' }}
-    - name: cache opam
+          if_true: "D:\\cygwin\\home\\runneradmin\\.opam"
+          if_false: "~/.opam"
+
+      - name: Checkout code
+        uses: actions/checkout@main
+
+      - name: Cache opam
         id: cache-opam
         uses: actions/cache@v2
         with:
-        path: _opam
+          path: ${{ steps.path.outputs.value }}
-        key: opam-${{matrix.operating-system}}-${{matrix.ocaml-version}}
+          key: opam-${{ matrix.os }}-${{ matrix.ocaml }}
+
+      - name: Use OCaml ${{ matrix.ocaml }}
+        uses: avsm/setup-ocaml@v1
+        with:
+          ocaml-version: ${{ matrix.ocaml }}
+
+      - run: opam update -u
+        if: steps.cache-opam.outputs.cache-hit == 'true'
+
       - run: opam pin -n .
-      if: steps.cache-opam.outputs.cache-hit != 'true'
+
       - run: opam depext -yt containers containers-data containers-thread
-      if: steps.cache-opam.outputs.cache-hit != 'true'
+
-    - run: opam install -t . --deps-only
+      - run: opam install -t containers containers-data containers-thread --deps-only
-      if: ${{ matrix.os == 'ubuntu-latest' }} and ${{ steps.cache-opam.outputs.cache-hit != 'true' }}
+        if: matrix.os == 'ubuntu-latest'
-    - run: opam install . --deps-only  # no test deps
+
-      if: ${{ matrix.os != 'ubuntu-latest' }} and ${{ steps.cache-opam.outputs.cache-hit != 'true' }}
+      - run: |
+          opam install -t containers --deps-only
+          opam install containers-data containers-thread --deps-only  # no test deps
+        if: matrix.os != 'ubuntu-latest'
+
       - run: opam exec -- dune build
+
       - run: opam exec -- dune runtest
         if: ${{ matrix.os == 'ubuntu-latest' }}
+
       - run: opam exec -- dune runtest -p containers # test only core on non-ubuntu platform
         if: ${{ matrix.os != 'ubuntu-latest' }}
+
+      - name: Remove troublesome files for caching on windows
+        if: matrix.os == 'windows-latest'
+        run: |
+            if (Test-Path D:\\cygwin\\home\\runneradmin\\.opam\\ocaml-variants.${{ matrix.ocaml }}+mingw64c\\.opam-switch\\sources\\ocamlbuild.0.14.0\\examples\\){Remove-Item –path D:\\cygwin\\home\\runneradmin\\.opam\\ocaml-variants.${{ matrix.ocaml }}+mingw64c\\.opam-switch\\sources\\ocamlbuild.0.14.0\\examples\\ –recurse}