Improve PR review checklist
還沒有人認領這個 Issue。
評估
- 難度
- 2/5
- 預估耗時
- 1-2 天
- 新手友好度
- 32/100
- Issue 類型
- 文件
- 描述清晰度
- 基本清楚
- 活躍度
- 停滯
- 技術堆疊
- github
研究方向
閱讀 pull-request-lifecycle 和 triaging 頁面中的「Reviewing」部分，然後檢查連結的 CPython pull request 留言中的檢查清單。將相關檢查清單整合到 pull-request-lifecycle 中，並移除 triaging 中重複的 Reviewing 指引；完成的標準是兩個頁面不再重複相同的資訊。
由索引模型根據 Issue 內容生成。
描述
In https://github.com/python/cpython/pull/93823#issuecomment-1156152572 @erlend-aasland wrote a checklist about reviewing PRs.
In https://github.com/python/devguide/pull/914#issuecomment-1207257987, I suggested:
SGTM -- that's a nice list. I think it would be easier to add it in a separate PR though.
Reviewing PRs is currently documented in two places:
- https://devguide.python.org/getting-started/pull-request-lifecycle/#reviewing
- https://devguide.python.org/triage/triaging/#reviewing-patches
The former has been likely written from scratch after the HG->Git migration, whereas the latter formerly documented how to review patches attached to a bpo issue and then it has been adapted to GitHub. I think the best approach would be to enhance the pull-request-lifecycle with your list, and remove the duplicated info in triaging.
- 主要語言
- Python
- 星號
- 2.1k
- 分支
- 1k
- 平均合併
- 2 天 12 小時
- 30 天內合併 PR
- 12
貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
python/devguide 的其他 Issue
-
type-feature
難度 2/5 1-3 小時 新手友好度 68/100
-
type-feature
難度 2/5 1-3 小時 新手友好度 68/100
-
topic-building python type-feature
難度 2/5 1-3 小時 新手友好度 65/100
-
needs: decision topic-test type-bug
難度 2/5 1-3 小時 新手友好度 78/100
-
topic-dev process type-feature
難度 2/5 1-3 小時 新手友好度 62/100
相似的 Issue
-
Add: hunch 未關閉
難度 2/5 1-3 小時 新手友好度 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
難度 2/5 1-3 小時 新手友好度 72/100
-
難度 2/5 1-3 小時 新手友好度 68/100
DiamondLightSource/dodal#2211 ·
-
難度 2/5 1-3 小時 新手友好度 88/100
openml/openml-python#1749 ·
-
難度 2/5 1-3 小時 新手友好度 88/100
sipyourdrink-ltd/bernstein#6191 ·