Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump upload-artifact and download-artifact action to v4 #652

Merged
merged 5 commits into from
Jan 16, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 18 additions & 5 deletions .github/workflows/packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,9 @@ jobs:
patchelf --replace-needed ${SQLITE_LIB} libsqlite3.so.0 ./pygeodiff-binaries/libpygeodiff-${GEODIFF_VER}-python.so
patchelf --remove-rpath ./pygeodiff-binaries/libpygeodiff-${GEODIFF_VER}-python.so

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: artifact-pygeodiff-linux
path: ./pygeodiff-binaries/*.so

build_windows_binaries:
Expand All @@ -75,8 +76,9 @@ jobs:
unzip -o pygeodiff-$env:GEODIFF_VER-cp$env:PYTHON_VER-cp$env:PYTHON_VER-win_amd64.whl -d tmp64
copy tmp64\pygeodiff\*.pyd pygeodiff-binaries\

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: artifact-pygeodiff-windows
path: ./pygeodiff-binaries/*.pyd

build_macos_binary:
Expand Down Expand Up @@ -109,8 +111,9 @@ jobs:
exit 1
fi

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
with:
name: artifact-pygeodiff-macos
path: ./pygeodiff-binaries/*.dylib

create_mergin_plugin_package:
Expand Down Expand Up @@ -147,8 +150,16 @@ jobs:
echo "geodiff version defined in python-api-client requires.txt $GEODIFF_VER_FROM_CLIENT does not equal $GEODIFF_VER from the workpackage file"
exit 1; # or just warning??
fi
- name: Merge Artifacts
uses: actions/upload-artifact/merge@v4
with:
name: artifact
path: artifact-pygeodiff
pattern: artifact-pygeodiff-*
merge-multiple: true
delete-merged: true

- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
name: artifact
path: pygeodiff-binaries
Expand All @@ -169,7 +180,9 @@ jobs:
cp qgis-mergin-plugin/LICENSE.txt output/Mergin/LICENSE
(cd output && zip -r9 ../mergin.zip Mergin/)

- uses: actions/upload-artifact@v3


- uses: actions/upload-artifact@v4
with:
name: Mergin
path: output/
Expand Down
Loading