Skip to content

Commit

Permalink
Update cmake-multi-platform.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
martinjankoehler authored Dec 5, 2024
1 parent baafd31 commit 6382539
Showing 1 changed file with 6 additions and 10 deletions.
16 changes: 6 additions & 10 deletions .github/workflows/cmake-multi-platform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ jobs:
if: success() || failure()
uses: actions/upload-artifact@v4
with:
name: upload-python-coverage-report-${{ matrix.os }}_${{ matrix.python-version }}
name: upload-python-coverage-report-${{ matrix.os }}_py${{ matrix.python-version }}
path: ${{ env.COVERAGE_REPORT_PATH }}
retention-days: 1
if-no-files-found: error
Expand All @@ -206,7 +206,7 @@ jobs:
if: success() || failure()
uses: actions/upload-artifact@v4
with:
name: upload-python-allure-report-${{ matrix.os }}_${{ matrix.python-version }}
name: upload-python-allure-report-${{ matrix.os }}_py${{ matrix.python-version }}
path: ${{ env.ALLURE_RESULTS_PATH }}
retention-days: 1
if-no-files-found: error
Expand All @@ -218,6 +218,8 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4

# NOTE: actions/download-artifact@v4 now has merging functionality built in => no need for actions/upload-artifact/merge@v4
# - name: "Merge test results (Allure)"
# uses: actions/upload-artifact/merge@v4
Expand Down Expand Up @@ -272,14 +274,8 @@ jobs:
runs-on: ubuntu-latest

steps:
# - name: "Merge test results (Python Coverage)"
# uses: actions/upload-artifact/merge@v4
# with:
# name: merged-python-coverage-reports
# pattern: upload-python-coverage-report-*
# # delete-merged: true
# retention-days: 1

- uses: actions/checkout@v4

- name: "Download and merge Python coverage artifacts"
uses: actions/download-artifact@v4
with:
Expand Down

0 comments on commit 6382539

Please sign in to comment.