-
Notifications
You must be signed in to change notification settings - Fork 13.4k
PR for llvm/llvm-project#80348 #80585
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
Conversation
Modifying a cherry-picked patch to fix code formatting issues can be risky, so we don't typically do this. Therefore, it's not necessary to run this job on the release branches. (cherry picked from commit 2193c95)
@tru What do you think about merging this PR to the release branch? |
@llvm/pr-subscribers-github-workflow Author: None (llvmbot) Changesresolves llvm/llvm-project#80348 Full diff: https://github.com/llvm/llvm-project/pull/80585.diff 1 Files Affected:
diff --git a/.github/workflows/pr-code-format.yml b/.github/workflows/pr-code-format.yml
index 5223089ee8a93..c8fd541fd34db 100644
--- a/.github/workflows/pr-code-format.yml
+++ b/.github/workflows/pr-code-format.yml
@@ -1,5 +1,8 @@
name: "Check code formatting"
on: pull_request_target
+ branches:
+ - main
+
permissions:
pull-requests: write
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is incorrectly formatted (sorry for not catching that in my initial review).
It should be the following:
on:
pull_request_target:
branches:
- main
I pushed a fixup to main 4b34558.
/cherry-pick 4b34558. |
Failed to cherry-pick: 4b34558. https://github.com/llvm/llvm-project/actions/runs/7789532649 Please manually backport the fix and push it to your github fork. Once this is done, please create a pull request |
/pull-request #80776 |
resolves #80348