more

aryx · Apr 19, 2024 · ecd708f · ecd708f
1 parent 320de22
commit ecd708f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/build-and-test.jsonnet b/.github/workflows/build-and-test.jsonnet
@@ -22,12 +22,12 @@ local job = {
 	//TODO: 'windows-latest'
 	],
       'ocaml-compiler': [
-	// Version that I ported ocamlrun to plan9 (need stdcompat
-	// so we can use |> and bytes type without issues)
+	// Old OCaml version where I ported ocamlrun to plan9
+	// This needs stdcompat so we can use |> and bytes type without issues.
+	// The |> operator was introduced in 4.02.0, that we could add in
+	// the matrix, but stdcompat does not compile with it.
 	'3.10.0',
-	// Old version with |> builtin
-	'4.02.1',
-	// first version with valid ocamlformat
+	// First OCaml version with a working ocamlformat OPAM package
 	'4.04.1',
 	//TODO: Ideally 4.14.1 and ocaml latest (5.2.0)
 	],

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -22,7 +22,6 @@ jobs:
       matrix:
         ocaml-compiler:
           - 3.10.0
-          - 4.02.1
           - 4.04.1
         os:
           - ubuntu-latest