[llvm] [libc++][ci] Use upload-artifact at v4 instead of @v3 (PR #79854)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 29 08:06:17 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-github-workflow
Author: Louis Dionne (ldionne)
<details>
<summary>Changes</summary>
The upload-artifact@<!-- -->v3 action is using Node 16, which is reaching EOL. As a result, we are getting warnings prompting us to move our jobs over to the latest version of upload-artifact.
---
Full diff: https://github.com/llvm/llvm-project/pull/79854.diff
1 Files Affected:
- (modified) .github/workflows/libcxx-build-and-test.yaml (+3-3)
``````````diff
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index 9579c05f1c5b1af..ba5a63eac8bf8b4 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -75,7 +75,7 @@ jobs:
CC: ${{ matrix.cc }}
CXX: ${{ matrix.cxx }}
ENABLE_CLANG_TIDY: ${{ matrix.clang_tidy }}
- - uses: actions/upload-artifact at v3
+ - uses: actions/upload-artifact at v4
if: always()
with:
name: ${{ matrix.config }}-${{ matrix.cxx }}-results
@@ -124,7 +124,7 @@ jobs:
CC: ${{ matrix.cc }}
CXX: ${{ matrix.cxx }}
ENABLE_CLANG_TIDY: ${{ matrix.clang_tidy }}
- - uses: actions/upload-artifact at v3
+ - uses: actions/upload-artifact at v4
if: always() # Upload artifacts even if the build or test suite fails
with:
name: ${{ matrix.config }}-results
@@ -189,7 +189,7 @@ jobs:
CC: clang-18
CXX: clang++-18
ENABLE_CLANG_TIDY: "OFF"
- - uses: actions/upload-artifact at v3
+ - uses: actions/upload-artifact at v4
if: always()
with:
name: ${{ matrix.config }}-results
``````````
</details>
https://github.com/llvm/llvm-project/pull/79854
More information about the llvm-commits
mailing list