Skip to content

Commit

Permalink
fix: Fix GitHub action artifacts after update to v4 (#303)
Browse files Browse the repository at this point in the history
  • Loading branch information
mkb79 authored Jan 23, 2024
1 parent 3f43f8f commit 1bc9198
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ jobs:
poetry build --ansi
- name: Upload build package artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: dist
path: dist
Expand All @@ -84,7 +84,7 @@ jobs:
id-token: write
steps:
- name: Download build package artifact
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: dist
path: dist
Expand Down
13 changes: 8 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -97,14 +97,14 @@ jobs:
- name: Upload coverage data
if: always() && matrix.session == 'tests'
uses: "actions/upload-artifact@v3"
uses: "actions/upload-artifact@v4"
with:
name: coverage-data
name: coverage-data-${{ matrix.python }}-${{ matrix.os }}
path: ".coverage.*"

- name: Upload documentation
if: matrix.session == 'docs-build'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: docs
path: docs/_build
Expand Down Expand Up @@ -138,9 +138,10 @@ jobs:
nox --version
- name: Download coverage data
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: coverage-data
pattern: coverage-data-*
merge-multiple: true

- name: Combine coverage data and display human readable report
run: |
Expand All @@ -152,3 +153,5 @@ jobs:
- name: Upload coverage report
uses: codecov/codecov-action@v3.1.4
with:
files: ./coverage.xml,./coverage1.xml,./coverage2.xml

0 comments on commit 1bc9198

Please sign in to comment.