Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 964579e

Browse files
committedNov 6, 2022
CI stuff for syntax
1 parent cf5fa9c commit 964579e

File tree

2 files changed

+41
-7
lines changed

2 files changed

+41
-7
lines changed
 

‎.github/workflows/ci.yml

+40-6
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,6 @@ jobs:
3030
3131
- name: Checkout
3232
uses: actions/checkout@v3
33-
with:
34-
submodules: true
3533

3634
- name: Use OCaml ${{matrix.ocaml_compiler}}
3735
uses: ocaml/setup-ocaml@v2
@@ -40,10 +38,41 @@ jobs:
4038
opam-pin: false
4139
opam-depext: false
4240

43-
- name: Use Node.js
44-
uses: actions/setup-node@v3
45-
with:
46-
node-version: 16
41+
# grep: out of memory
42+
# /Users/ostera/rescript-org-actions-runner/_work/_actions/tj-actions/changed-files/v34/get-changed-paths.sh: line 66: echo: write error: Broken pipe
43+
# Error: Process completed with exit code 1.
44+
# - name: "Check if syntax subfolder has changed"
45+
# id: syntax-changed
46+
# uses: tj-actions/changed-files@v34
47+
# with:
48+
# files: syntax/**
49+
50+
# - name: List all changed files
51+
# shell: bash
52+
# run: |
53+
# for file in ${{ steps.syntax-changed.outputs.all_changed_files }}; do
54+
# echo "$file was changed"
55+
# done
56+
57+
- name: "Syntax: install OPAM dependencies"
58+
run: opam install . --deps-only
59+
working-directory: syntax
60+
61+
- name: "Syntax: build CLI"
62+
run: opam exec -- dune build
63+
working-directory: syntax
64+
65+
- name: "Syntax: Run roundtrip tests"
66+
if: runner.os != 'Windows'
67+
# if: ${{ runner.os != 'Windows' && steps.syntax-changed-specific.outputs.any_changed == 'true' }}
68+
run: opam exec -- make roundtrip-test
69+
working-directory: syntax
70+
71+
- name: "Syntax: Run tests (Windows)"
72+
if: runner.os == 'Windows'
73+
# if: ${{ runner.os == 'Windows' && steps.syntax-changed-specific.outputs.any_changed == 'true' }}
74+
run: opam exec -- make test
75+
working-directory: syntax
4776

4877
# Required for ninja build
4978
- name: "Windows: Use MSVC"
@@ -55,6 +84,11 @@ jobs:
5584
- name: Build ninja
5685
run: node scripts/buildNinjaBinary.js
5786

87+
- name: Use Node.js
88+
uses: actions/setup-node@v3
89+
with:
90+
node-version: 16
91+
5892
- name: NPM install
5993
run: opam exec -- npm ci
6094
env:

‎dune

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
(dirs jscomp)
1+
(dirs jscomp syntax)

0 commit comments

Comments
 (0)
Please sign in to comment.