Skip to content

Commit

Permalink
github/workflows: fix artifact upload step order
Browse files Browse the repository at this point in the history
It must execute before the others in order for the interpreter
to be uploaded even if failures occur when building the tools,
the tests or while running the test suite.
  • Loading branch information
matheusmoreira committed Sep 21, 2024
1 parent 21861fe commit 72e3541
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/lone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,14 @@ jobs:
- name: Build lone
run: make --jobs --output-sync lone

- name: Upload lone interpreter artifact
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.compiler.name }}-${{ matrix.linker.name }}-lone
path: build/${{ matrix.compiler.name }}-${{ matrix.linker.name }}/lone
compression-level: 9
overwrite: false

- name: Build lone's tools
run: make --jobs --output-sync tools

Expand All @@ -71,14 +79,6 @@ jobs:
- name: Test lone
run: make test

- name: Upload lone executable artifact
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.compiler.name }}-${{ matrix.linker.name }}-lone
path: build/${{ matrix.compiler.name }}-${{ matrix.linker.name }}/lone
compression-level: 9
overwrite: false

analyze:
name: CodeQL analysis
runs-on: ubuntu-latest
Expand Down

0 comments on commit 72e3541

Please sign in to comment.