Skip to content

Commit

Permalink
Use latest upload/download actions (#1410)
Browse files Browse the repository at this point in the history
  • Loading branch information
jsoref committed Apr 5, 2024
1 parent 17d095a commit e03bdc4
Showing 1 changed file with 15 additions and 13 deletions.
28 changes: 15 additions & 13 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,9 @@ jobs:
cmd /c "$env:VSDevCmd" "&" msbuild /m /clp:ForceConsoleColor "$env:msbuild_config_props" cppwinrt.sln /t:cppwinrt
- name: Upload built executables
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
name: msvc-build-${{ matrix.compiler}}-${{ matrix.arch }}-${{ matrix.config }}-bin
path: |
_build/${{ matrix.arch }}/${{ matrix.config }}/*.exe
_build/${{ matrix.arch }}/${{ matrix.config }}/*.dll
Expand Down Expand Up @@ -116,16 +116,16 @@ jobs:

- name: Fetch cppwinrt executables
if: matrix.arch != 'arm64'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
name: msvc-build-${{ matrix.compiler}}-${{ matrix.arch }}-${{ matrix.config }}-bin
path: _build/${{ matrix.arch }}/${{ matrix.config }}/

- name: Fetch x86 cppwinrt executables (arm64 only)
if: matrix.arch == 'arm64'
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: msvc-build-x86-Release-bin
name: msvc-build-${{ matrix.compiler}}-x86-Release-bin
path: _build/x86/Release/

- name: Download nuget
Expand Down Expand Up @@ -240,9 +240,9 @@ jobs:
- name: Upload arm64 test executables
if: matrix.arch == 'arm64'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: msvc-tests-${{ matrix.arch }}-${{ matrix.config }}-bin
name: msvc-tests-${{ matrix.test_exe }}-${{ matrix.compiler }}-${{ matrix.arch }}-${{ matrix.config }}-bin
path: |
_build/${{ matrix.arch }}/${{ matrix.config }}/*.exe
_build/${{ matrix.arch }}/${{ matrix.config }}/*.dll
Expand Down Expand Up @@ -279,7 +279,7 @@ jobs:
cmake --build build/cross_x64/ --target install -j2
- name: Upload cppwinrt.exe
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: cross-build-${{ matrix.arch }}-bin
path: install/bin/cppwinrt.exe
Expand Down Expand Up @@ -323,20 +323,22 @@ jobs:
cmd /c "$env:VSDevCmd" "&" msbuild /m /clp:ForceConsoleColor "$env:msbuild_config_props" /p:Deployment=${{ matrix.Deployment }} natvis\cppwinrtvisualizer.sln
build-msvc-nuget-test:
name: 'Build nuget test'
name: 'Build nuget test (${{ matrix.arch }})'
needs: test-msvc-cppwinrt-build
strategy:
matrix:
compiler:
- MSVC
arch: [x86, x64]
config: [Release]
runs-on: windows-latest
steps:
- uses: actions/checkout@v4

- name: Fetch cppwinrt executables
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: msvc-build-${{ matrix.arch }}-${{ matrix.config }}-bin
name: msvc-build-${{ matrix.compiler}}-${{ matrix.arch }}-${{ matrix.config }}-bin
path: _build/${{ matrix.arch }}/${{ matrix.config }}/

- name: Download nuget
Expand Down Expand Up @@ -396,7 +398,7 @@ jobs:
}
- name: Upload nuget package artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: package
path: "*.nupkg"

0 comments on commit e03bdc4

Please sign in to comment.