Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

Since this is a composite action, we do not get any testing currently
when updating the action. This patch adds a simple workflow to test the
action so we can ensure we do not break it when modifying it.

Created using spr 1.3.7
@llvmbot
Copy link
Member

llvmbot commented Nov 11, 2025

@llvm/pr-subscribers-github-workflow

Author: Aiden Grossman (boomanaiden154)

Changes

Since this is a composite action, we do not get any testing currently
when updating the action. This patch adds a simple workflow to test the
action so we can ensure we do not break it when modifying it.


Full diff: https://github.com/llvm/llvm-project/pull/167434.diff

1 Files Affected:

  • (added) .github/workflows/test-unprivileged-download-artifact.yml (+47)
diff --git a/.github/workflows/test-unprivileged-download-artifact.yml b/.github/workflows/test-unprivileged-download-artifact.yml
new file mode 100644
index 0000000000000..cb37cfebf1dc9
--- /dev/null
+++ b/.github/workflows/test-unprivileged-download-artifact.yml
@@ -0,0 +1,47 @@
+name: Test Unprivileged Download Artifact Action
+
+permissions:
+  contents: read
+
+on:
+  push:
+    branches:
+      - main
+    paths:
+      - .github/workflows/test-unprivileged-download-artifact.yml
+      - '.github/workflows/unprivileged-download-artifact/**'
+  pull_request:
+    paths:
+      - .github/workflows/test-unprivileged-download-artifact.yml
+      - '.github/workflows/unprivileged-download-artifact/**'
+
+jobs:
+  test-unprivileged-download-artifact:
+    name: Test Unprivileged Download Artifact
+    if: github.repository_owner == 'llvm'
+    runs-on: ubuntu-24.04
+    steps:
+      - name: Checkout LLVM
+        uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+        with:
+          sparse-checkout: |
+            .github/workflows/
+      - name: Create Test File
+        run: |
+          echo "test" > comment
+      - name: Upload Test File
+        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
+        with:
+          name: workflow-args
+          path: |
+            comments
+      - name: Download Artifact
+        uses: ./.github/workflows/unprivileged-download-artifact
+        id: download-artifact
+        with:
+          run-id: ${{ github.run_id }}
+          artifact-name: workflow-args
+      - name: Assert That Contents are the Same
+        run: |
+          cat test
+          cat workflow-args

Created using spr 1.3.7
Created using spr 1.3.7
@boomanaiden154 boomanaiden154 marked this pull request as draft November 11, 2025 01:40
Created using spr 1.3.7
Created using spr 1.3.7
Created using spr 1.3.7
Created using spr 1.3.7
Created using spr 1.3.7
@boomanaiden154 boomanaiden154 marked this pull request as ready for review November 11, 2025 02:11
@boomanaiden154 boomanaiden154 merged commit cdc6f85 into main Nov 11, 2025
16 checks passed
@boomanaiden154 boomanaiden154 deleted the users/boomanaiden154/github-add-workflow-to-test-unprivileged-download-artifact-action branch November 11, 2025 02:19
llvm-sync bot pushed a commit to arm/arm-toolchain that referenced this pull request Nov 11, 2025
…ct Action

Since this is a composite action, we do not get any testing currently
when updating the action. This patch adds a simple workflow to test the
action so we can ensure we do not break it when modifying it.

Reviewers: tstellar

Reviewed By: tstellar

Pull Request: llvm/llvm-project#167434
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants