Skip to content

Commit

Permalink
artifact naming is hard
Browse files Browse the repository at this point in the history
  • Loading branch information
bitsandfoxes committed Dec 20, 2024
1 parent 3eceb33 commit 9294b63
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/android-smoke-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,12 @@ on:

jobs:
run:
name: Android Smoke Test
name: Android Smoke Test ${{ matrix.init-type }}
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
init-type: ["runtime", "buildtime"]
env:
ARTIFACTS_PATH: samples/IntegrationTest/test-artifacts/
HOMEBREW_NO_INSTALL_CLEANUP: 1
Expand All @@ -33,7 +37,7 @@ jobs:
- name: Download test app artifact
uses: actions/download-artifact@v4
with:
name: testapp-Android-compiled-${{ inputs.unity-version }}
name: testapp-Android-compiled-${{ inputs.unity-version }}-${{ matrix.init-type }}
path: samples/IntegrationTest/Build

# See https://github.blog/changelog/2023-02-23-hardware-accelerated-android-virtualization-on-actions-windows-and-linux-larger-hosted-runners/
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -552,7 +552,7 @@ jobs:
if: ${{ failure() }}
uses: actions/upload-artifact@v4
with:
name: failed-project-${{ matrix.platform }}-${{ matrix.unity-version }}-but-compiled
name: failed-project-${{ matrix.platform }}-${{ matrix.unity-version }}-${{ matrix.init-type }}-but-compiled
path: |
samples/IntegrationTest
!samples/IntegrationTest/Build/*_BackUpThisFolder_ButDontShipItWithYourGame
Expand Down Expand Up @@ -600,7 +600,7 @@ jobs:
- name: Download app artifact
uses: actions/download-artifact@v4
with:
name: testapp-iOS-compiled-${{ matrix.unity-version }}
name: testapp-iOS-compiled-${{ matrix.unity-version }}-${{ matrix.init-type }}
path: samples/IntegrationTest/Build

- name: Set Xcode for iOS version ${{matrix.ios}}
Expand Down

0 comments on commit 9294b63

Please sign in to comment.