Skip to content

workflows/release-binaries-all: Add missing secret input #126921

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

tstellar
Copy link
Collaborator

Since d194c6b this workflow was missing the secret input which was causing it to fail.

Since d194c6b this workflow was missing
the secret inputw which was causing it to fail.
@llvmbot
Copy link
Member

llvmbot commented Feb 12, 2025

@llvm/pr-subscribers-github-workflow

Author: Tom Stellard (tstellar)

Changes

Since d194c6b this workflow was missing the secret input which was causing it to fail.


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

1 Files Affected:

  • (modified) .github/workflows/release-binaries-all.yml (+4)
diff --git a/.github/workflows/release-binaries-all.yml b/.github/workflows/release-binaries-all.yml
index d18b9b0b5c2ff..fd4694ebea32d 100644
--- a/.github/workflows/release-binaries-all.yml
+++ b/.github/workflows/release-binaries-all.yml
@@ -27,6 +27,10 @@ on:
         required: true
         default: false
         type: boolean
+    secrets:
+      RELEASE_TASKS_USER_TOKEN:
+        description: "Secret used to check user permissions."
+        required: false
 
   pull_request:
     types:

@tstellar tstellar merged commit a684e0e into llvm:main Feb 12, 2025
16 of 20 checks passed
flovent pushed a commit to flovent/llvm-project that referenced this pull request Feb 13, 2025
Since d194c6b this workflow was missing
the secret input which was causing it to fail.
joaosaffran pushed a commit to joaosaffran/llvm-project that referenced this pull request Feb 14, 2025
Since d194c6b this workflow was missing
the secret input which was causing it to fail.
sivan-shani pushed a commit to sivan-shani/llvm-project that referenced this pull request Feb 24, 2025
Since d194c6b this workflow was missing
the secret input which was causing it to fail.
@tstellar tstellar added this to the LLVM 20.X Release milestone Feb 26, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Feb 26, 2025
@tstellar
Copy link
Collaborator Author

/cherry-pick a684e0e

@llvmbot
Copy link
Member

llvmbot commented Feb 26, 2025

/pull-request #128884

@llvmbot llvmbot moved this from Needs Triage to Done in LLVM Release Status Feb 26, 2025
swift-ci pushed a commit to swiftlang/llvm-project that referenced this pull request Mar 4, 2025
Since d194c6b this workflow was missing
the secret input which was causing it to fail.

(cherry picked from commit a684e0e)
sabitov-kirill added a commit to sabitov-kirill/llvm-project that referenced this pull request Apr 2, 2025
* workflows/release-binaries-all: Add missing secret input (llvm#126921)

Since d194c6b this workflow was missing
the secret input which was causing it to fail.

(cherry picked from commit a684e0e)

* workflows/release-binaries: Fix macos-14 build (llvm#127157)

This was broken when pgo was enabled by
0572580.

(cherry picked from commit d595d5a)

* [clang-format] Fix a bug that changes keyword `or` to an identifier (llvm#128410)

Backports ffc61dc 0968df9 2d585cc

Fixes llvm#105482

* [clangd] Add clangd 20 release notes (llvm#127358)

* Bump version to 20.1.0 (final)

* Add dummy "SystemS" target

* Add barebone SystemS backend initislisation, that can succsesfully perform`CodeGenTargetMachineImpl::initAsmInfo`.

Commit includes simplest form of registers, instructions, subtarget descrption.

Commit includes simple lit test to check how far we can get in performing end-to-end compilation pipeline.

* Add functional backend, that can emit basical asm

* Add different testing scenarious

* Add dag selection to pass config
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