Skip to content

Also move syntax_tests under tests #7090

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 8, 2024
Merged

Also move syntax_tests under tests #7090

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fc0cc7b
Remove ml syntax tests
cknitt Oct 5, 2024
ba3a92a
compiler/syntax/tests -> tests/syntax_tests
cknitt Oct 7, 2024
4dc732a
Adapt scripts
cknitt Oct 7, 2024
6dbd99a
Always run the syntax tests
cknitt Oct 7, 2024
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Always run the syntax tests
  • Loading branch information
@cknitt
cknitt committed Oct 7, 2024
commit 6dbd99a49e0405affd72efcd0a9b2e40548d81a2
16 changes: 2 additions & 14 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,6 @@ jobs:

- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 2 # to be able to check for changes in subfolder compiler/syntax later

- name: Use Node.js
uses: actions/setup-node@v4
Expand Down Expand Up @@ -297,22 +295,12 @@ jobs:
if: steps.ninja-build-cache.outputs.cache-hit != 'true'
run: node ./scripts/copyExes.js -ninja

- name: "Check if syntax subfolder has changed"
id: syntax-diff
shell: bash
run: |
if git diff --name-only --exit-code HEAD^ HEAD -- compiler/syntax; then
echo "syntax_status=unchanged" >> $GITHUB_ENV
else
echo "syntax_status=changed" >> $GITHUB_ENV
fi

- name: "Syntax: Run roundtrip tests"
if: ${{ env.syntax_status == 'changed' && runner.os != 'Windows' }}
if: ${{ runner.os != 'Windows' }}
run: opam exec -- make test-syntax-roundtrip

- name: "Syntax: Run tests (Windows)"
if: ${{ env.syntax_status == 'changed' && runner.os == 'Windows' }}
if: ${{ runner.os == 'Windows' }}
run: opam exec -- make test-syntax

- name: Build runtime/stdlib
Expand Down