CI stuff for syntax

Author: cknitt

.github/workflows/ci.yml

@@ -31,7 +31,7 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v3
         with:
-          submodules: true
+          fetch-depth: 2 # to be able to check for changes in subfolder res_syntax later
 
       - name: Use OCaml ${{matrix.ocaml_compiler}}
         uses: ocaml/setup-ocaml@v2
@@ -40,10 +40,35 @@ jobs:
           opam-pin: false
           opam-depext: false
 
-      - name: Use Node.js
-        uses: actions/setup-node@v3
-        with:
-          node-version: 16
+      - name: "Check if syntax subfolder has changed"
+        id: syntax-diff
+        shell: bash
+        run: |
+          if git diff --name-only --exit-code HEAD^ HEAD -- res_syntax; then
+            echo "syntax_status=unchanged" >> $GITHUB_ENV
+          else
+            echo "syntax_status=changed" >> $GITHUB_ENV
+          fi
+
+      - name: "Syntax: install OPAM dependencies"
+        if: env.syntax_status == 'changed'
+        run: opam install . --deps-only
+        working-directory: res_syntax
+
+      - name: "Syntax: build CLI"
+        if: env.syntax_status == 'changed'
+        run: opam exec -- dune build
+        working-directory: res_syntax
+
+      - name: "Syntax: Run roundtrip tests"
+        if: ${{ env.syntax_status == 'changed' && runner.os != 'Windows' }}
+        run: opam exec -- make roundtrip-test
+        working-directory: res_syntax
+
+      - name: "Syntax: Run tests (Windows)"
+        if: ${{ env.syntax_status == 'changed' && runner.os == 'Windows' }}
+        run: opam exec -- make test
+        working-directory: res_syntax
 
       # Required for ninja build
       - name: "Windows: Use MSVC"
@@ -55,6 +80,11 @@ jobs:
       - name: Build ninja
         run: node scripts/buildNinjaBinary.js
 
+      - name: Use Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16
+
       - name: NPM install
         run: opam exec -- npm ci
         env:

.prettierignore

@@ -2,7 +2,7 @@ jscomp/
 lib/
 vendor/
 ninja/
-syntax/
+res_syntax/
 _opam
 CHANGELOG.md
 README.md

dune

@@ -1 +1 @@
-(dirs jscomp)
+(dirs jscomp res_syntax)

scripts/ninja.js

@@ -1454,16 +1454,16 @@ function setSortedToStringAsNativeDeps(xs) {
  */
 function preprocessorNinjaSync() {
   var napkinFiles = fs
-    .readdirSync(path.join(jscompDir, "..", "syntax", "src"), "ascii")
+    .readdirSync(path.join(jscompDir, "..", "res_syntax", "src"), "ascii")
     .filter(x => x.endsWith(".ml") || x.endsWith(".mli"));
   var napkinCliFiles = fs
-    .readdirSync(path.join(jscompDir, "..", "syntax", "cli"), "ascii")
+    .readdirSync(path.join(jscompDir, "..", "res_syntax", "cli"), "ascii")
     .filter(x => x.endsWith(".ml") || x.endsWith(".mli"));
   var buildNapkinFiles = napkinFiles
-    .map(file => `o napkin/${file} : copy ../syntax/src/${file}`)
+    .map(file => `o napkin/${file} : copy ../res_syntax/src/${file}`)
     .join("\n");
   var buildNapkinCliFiles = napkinCliFiles
-    .map(file => `o napkin/${file} : copy ../syntax/cli/${file}`)
+    .map(file => `o napkin/${file} : copy ../res_syntax/cli/${file}`)
     .join("\n");
   var cppoNative = `
 ocamlopt = ocamlopt.opt
@@ -1569,7 +1569,7 @@ rule copy
   description = $in -> $out    
 ${buildNapkinFiles}    
 ${buildNapkinCliFiles}
-o napkin/CHANGELOG.md : copy ../syntax/CHANGELOG.md
+o napkin/CHANGELOG.md : copy ../res_syntax/CHANGELOG.md
 `;
   var cppoNinjaFile = "cppoVendor.ninja";
   writeFileSync(path.join(jscompDir, cppoNinjaFile), cppoNative);