Skip to content

Conversation

llvmbot
Copy link
Member

@llvmbot llvmbot commented Oct 8, 2025

Backport 348ffe8

Requested by: @boomanaiden154

Fixes #162463.

We have a list of people authorized to create release assets to prevent
someone adding a release asset to a release without someone getting
alerted. Add dyung and c-rhodes now that they are also release managers.

Fixes llvm#162463 (maybe after backport).

(cherry picked from commit 348ffe8)
@llvmbot
Copy link
Member Author

llvmbot commented Oct 8, 2025

@tru What do you think about merging this PR to the release branch?

@llvmbot
Copy link
Member Author

llvmbot commented Oct 8, 2025

@llvm/pr-subscribers-github-workflow

Author: None (llvmbot)

Changes

Backport 348ffe8

Requested by: @boomanaiden154


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

1 Files Affected:

  • (modified) .github/workflows/release-asset-audit.py (+2)
diff --git a/.github/workflows/release-asset-audit.py b/.github/workflows/release-asset-audit.py
index 23b901a476dc0..0cff6c695921f 100644
--- a/.github/workflows/release-asset-audit.py
+++ b/.github/workflows/release-asset-audit.py
@@ -54,6 +54,8 @@ def _get_uploaders(release_version):
                 "tru",
                 "tstellar",
                 "github-actions[bot]",
+                "c-rhodes",
+                "dyung",
             ]
         )
 

@github-project-automation github-project-automation bot moved this from Needs Triage to Needs Merge in LLVM Release Status Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Merge
Development

Successfully merging this pull request may close these issues.

3 participants