Skip to content

workflows/release-documentation: Add missing checkout - #211082

Merged
tstellar merged 1 commit into
llvm:mainfrom
tstellar:doc-fix
Jul 22, 2026
Merged

tstellar merged 1 commit into
llvm:mainfrom
tstellar:doc-fix

Conversation

@tstellar

Copy link
Copy Markdown
Contributor

We need to checkout the upload-release-artifact composite action before using it.

We need to checkout the upload-release-artifact composite action before
using it.
@llvmorg-github-actions

Copy link
Copy Markdown

@llvm/pr-subscribers-github-workflow

Author: Tom Stellard (tstellar)

Changes

We need to checkout the upload-release-artifact composite action before using it.


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

1 Files Affected:

  • (modified) .github/workflows/release-documentation.yml (+7)
diff --git a/.github/workflows/release-documentation.yml b/.github/workflows/release-documentation.yml
index aa6aa7bc4f8f4..ce78178a43544 100644
--- a/.github/workflows/release-documentation.yml
+++ b/.github/workflows/release-documentation.yml
@@ -199,6 +199,13 @@ jobs:
       attestations: write # For artifact attestations
 
     steps:
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
+        with:
+          persist-credentials: false
+          sparse-checkout: |
+            .github/workflows/upload-release-artifact
+          sparse-checkout-cone-mode: false
+
      - name: Upload Man Page Artifacts
        id: man-page-artifact-upload
        uses: ./.github/workflows/upload-release-artifact

@boomanaiden154 boomanaiden154 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tstellar
tstellar merged commit 8ad500f into llvm:main Jul 22, 2026
16 of 17 checks passed
@tstellar tstellar added this to the LLVM 23.x Release milestone Jul 22, 2026
@github-project-automation github-project-automation Bot moved this to Needs Triage in LLVM Release Status Jul 22, 2026
@github-project-automation github-project-automation Bot moved this from Needs Triage to Done in LLVM Release Status Jul 22, 2026
@tstellar

Copy link
Copy Markdown
Contributor Author

/cherry-pick 8ad500f

@cenewcombe

cenewcombe commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

I'm not sure if this commit is causing issues, or if the CI hasn't picked up the change yet. I am seeing that workflows could not run on my most recent PR merge - commit # 3486c5d

The error references invalid syntax at line 186 of the yaml file.

@llvmbot

llvmbot commented Jul 22, 2026

Copy link
Copy Markdown
Member

/pull-request #211370

dyung pushed a commit that referenced this pull request Jul 23, 2026
We need to checkout the upload-release-artifact composite action before
using it.

(cherry picked from commit 8ad500f)
midhuncodes7 pushed a commit to midhuncodes7/llvm-project that referenced this pull request Jul 28, 2026
We need to checkout the upload-release-artifact composite action before
using it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants