Skip to content

Commit

Permalink
Don't name folder the same as file
Browse files Browse the repository at this point in the history
  • Loading branch information
juliannguyen4 committed Dec 1, 2023
1 parent 39abdbf commit c7a1bac
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
8 changes: 4 additions & 4 deletions .github/actions/set-matrix-output/action.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name: 'Set matrix output'
description: 'Use artifacts as a workaround for matrix outputs not being supported'
inputs:
file-name:
artifact-name:
required: true
test-outcome:
required: true
Expand All @@ -10,11 +10,11 @@ runs:
using: "composite"
steps:
- name: Save output file
run: echo ${{ inputs.test-outcome }} > ${{ inputs.file-name }}
run: echo ${{ inputs.test-outcome }} > ${{ inputs.artifact-name }}.txt
shell: bash

- name: Upload output file
uses: actions/upload-artifact@v3
with:
name: ${{ inputs.file-name }}
path: ${{ inputs.file-name }}
name: ${{ inputs.artifact-name }}
path: ${{ inputs.artifact-name }}.txt
10 changes: 5 additions & 5 deletions .github/workflows/stage-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ jobs:
if: ${{ inputs.commit_sha != '' && !cancelled() }}
uses: ./.github/actions/set-matrix-output
with:
file-name: sdist.${{ matrix.distros[1] }}.txt
artifact-name: sdist.${{ matrix.distros[1] }}
test-outcome: ${{ steps.test.outcome }}

# We also run some wheel (QE test category 2) tests
Expand Down Expand Up @@ -189,7 +189,7 @@ jobs:
if: ${{ inputs.commit_sha != '' && !cancelled() }}
uses: ./.github/actions/set-matrix-output
with:
file-name: ${{ matrix.python-version[1] }}-manylinux_${{ matrix.platform[0] }}.${{ matrix.distros[1] }}.txt
artifact-name: ${{ matrix.python-version[1] }}-manylinux_${{ matrix.platform[0] }}.${{ matrix.distros[1] }}
test-outcome: ${{ steps.test.outcome }}

manylinux_debian:
Expand Down Expand Up @@ -258,7 +258,7 @@ jobs:
if: ${{ inputs.commit_sha != '' && !cancelled() }}
uses: ./.github/actions/set-matrix-output
with:
file-name: ${{ matrix.python-version[1] }}-manylinux_${{ matrix.platform[0] }}.${{ matrix.debian-name }}.txt
artifact-name: ${{ matrix.python-version[1] }}-manylinux_${{ matrix.platform[0] }}.${{ matrix.debian-name }}
test-outcome: ${{ steps.test.outcome }}

manylinux_ubuntu2204:
Expand Down Expand Up @@ -322,7 +322,7 @@ jobs:
if: ${{ inputs.commit_sha != '' && !cancelled() }}
uses: ./.github/actions/set-matrix-output
with:
file-name: cp310-manylinux_${{ matrix.platform[0] }}.ubuntu2204.txt
artifact-name: cp310-manylinux_${{ matrix.platform[0] }}.ubuntu2204
test-outcome: ${{ steps.test.outcome }}

macOS:
Expand Down Expand Up @@ -387,5 +387,5 @@ jobs:
if: ${{ inputs.commit_sha != '' && !cancelled() }}
uses: ./.github/actions/set-matrix-output
with:
file-name: ${{ matrix.python-version[1] }}-macosx_x86_64.${{ matrix.macos-version }}.txt
artifact-name: ${{ matrix.python-version[1] }}-macosx_x86_64.${{ matrix.macos-version }}
test-outcome: ${{ steps.test.outcome }}

0 comments on commit c7a1bac

Please sign in to comment.