Skip to content

Commit

Permalink
workflows: use configuration for artifact upload
Browse files Browse the repository at this point in the history
Access the build directory programmatically via the configuration
name set in the environment of the current workflow in the matrix.
No wildcards necessary.

Also prepend the name of the current configuration
to the name of the uploaded artifact. This enables
the upload of all artifacts generated by the entire
build matrix since they'll all have different names.

Apparently there's no way to use the CONFIGURATION
environment variable from the matrix in this context,
leading to incredibly annoying repetition of the logic
that makes up the configuration name. Can't be helped.
  • Loading branch information
matheusmoreira committed Sep 21, 2024
1 parent 3fa90f6 commit 1485519
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/lone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ jobs:
- name: Upload lone executable artifact
uses: actions/upload-artifact@v4
with:
name: lone
path: build/*/lone
name: ${{ matrix.compiler.name }}-${{ matrix.linker.name }}-lone
path: build/${{ matrix.compiler.name }}-${{ matrix.linker.name }}/lone
compression-level: 9
overwrite: false

Expand Down

0 comments on commit 1485519

Please sign in to comment.