Assigning PRs to merger doesn't work when they're auto-merged
還沒有人認領這個 Issue。
評估
- 難度
- 3/5
- 預估耗時
- 1-2 天
- 新手友好度
- 35/100
研究方向
從 .github/update-release-branch.py 的第 163-168 行開始，並檢查範例 PR #2110，以了解 auto-merged pull requests 的指派方式是如何決定的。確認此情況下預期的 assignee 行為；完成標準是 auto-merged PRs 依照該規則指派，而不是錯誤地指派給 merger。
由索引模型根據 Issue 內容生成。
描述
A bunch of [
@jsoref's] PRs are assigned to the people who merged them. It feels like the automation hasn't contemplated that PRs would be written by people who can't/won't be the people who will click the "merge" button.
This is intentional:
https://github.com/github/codeql-action/blob/bc64d12bb9f349435efba65d373bac054665b85f/.github/update-release-branch.py#L163-L168
More interesting is that this approach seems to break when enabling auto-merge as can be seen here:
(merged by @angelapwen but PR is from @jsoref)
#2110 (@jsoref)
Originally posted by @intrigus-lgtm in https://github.com/github/codeql-action/issues/2131#issuecomment-1942529486
- 主要語言
- TypeScript
- 星號
- 1.6k
- 分支
- 493
- 平均合併
- 1 天 13 小時
- 30 天內合併 PR
- 44
貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
github/codeql-action 的其他 Issue
-
難度 2/5 1-3 小時 新手友好度 68/100
github/codeql-action#4052 · 4 則留言 ·
-
難度 4/5 3-5 天 新手友好度 45/100
github/codeql-action#4078 · 1 則留言 ·
-
難度 4/5 3-5 天 新手友好度 48/100
github/codeql-action#4008 · 9 則留言 ·
-
難度 4/5 3-5 天 新手友好度 52/100
github/codeql-action#3978 · 4 則留言 · 1 個 reaction ·
-
難度 3/5 1-2 天 新手友好度 48/100
github/codeql-action#3915 · 6 則留言 · 3 個 reaction ·
查看 github/codeql-action 的全部 Issue
相似的 Issue
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs 未關閉
難度 2/5 1-3 小時 新手友好度 70/100
-
Crush 未關閉
難度 1/5 1 小時以內 新手友好度 85/100
catppuccin/catppuccin#3125 ·
-
難度 1/5 1 小時以內 新手友好度 90/100
ElementsProject/cln-application#167 · 1 則留言 · 1 個 reaction ·
-
難度 2/5 1-3 小時 新手友好度 75/100
Quantco/pnpm-licenses#17 ·
-
難度 2/5 1-3 小時 新手友好度 75/100