Skip to content

Commit

Permalink
ci: cache fixtures
Browse files Browse the repository at this point in the history
Only generate fixtures if any grammar from any parser or the parser
generation itself has changed.
  • Loading branch information
dundargoc committed Feb 18, 2024
1 parent 48a1f12 commit 30ab745
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 8 deletions.
25 changes: 25 additions & 0 deletions .github/actions/cache/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
name: 'Cache'
description: "This action caches fixtures"
outputs:
cache-hit:
description: 'Cache hit'
value: ${{ steps.cache_output.outputs.cache-hit }}
runs:
using: "composite"
steps:
- uses: actions/cache@v4
id: cache_fixtures
with:
path: |
test/fixtures/grammars
target/release/tree-sitter-*.wasm
key: fixtures-${{ join(matrix.*, '_') }}-${{ hashFiles(
'.github/workflows/builds.yml',
'.github/workflows/sanitize.yml',
'cli/src/generate/**',
'script/generate-fixtures*',
'test/fixtures/grammars/*/**/src/*.c') }}

- run: echo "cache-hit=${{ steps.cache_fixtures.outputs.cache-hit }}" >> $GITHUB_OUTPUT
shell: bash
id: cache_output
11 changes: 6 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -156,16 +156,17 @@ jobs:
- name: Build CLI
run: $BUILD_CMD build --release --target=${{ matrix.target }} --features=${CLI_FEATURES}

- name: Fetch fixtures
if: ${{ !matrix.cli-only && inputs.run_test }} # Don't fetch fixtures for only CLI building targets
run: script/fetch-fixtures
- run: script/fetch-fixtures

- uses: ./.github/actions/cache
id: cache

- name: Generate fixtures
if: ${{ !matrix.cli-only && inputs.run_test }} # Can't natively run CLI on Github runner's host
if: ${{ !matrix.cli-only && inputs.run_test && steps.cache.outputs.cache-hit != 'true' }} # Can't natively run CLI on Github runner's host
run: script/generate-fixtures

- name: Generate WASM fixtures
if: ${{ !matrix.cli-only && !matrix.use-cross && inputs.run_test }} # See comment for the "Build wasm library" step
if: ${{ !matrix.cli-only && !matrix.use-cross && inputs.run_test && steps.cache.outputs.cache-hit != 'true' }} # See comment for the "Build wasm library" step
run: script/generate-fixtures-wasm

- name: Run main tests
Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/sanitize.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,12 @@ jobs:
- name: Build CLI
run: cargo build --release

- name: Fetch fixtures
run: script/fetch-fixtures
- run: script/fetch-fixtures

- name: Generate fixtures
- uses: ./.github/actions/cache
id: cache

- if: ${{ steps.cache.outputs.cache-hit != 'true' }}
run: script/generate-fixtures

- name: Run main tests with undefined behaviour sanitizer (UBSAN)
Expand Down

0 comments on commit 30ab745

Please sign in to comment.