Improve PR review checklist
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 32/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- github
- Lĩnh vực
- documentation
Hướng nghiên cứu
Đọc các phần “Reviewing” trong các trang pull-request-lifecycle và triaging, sau đó kiểm tra checklist trong bình luận pull request CPython được liên kết. Hợp nhất checklist liên quan vào pull-request-lifecycle và xóa hướng dẫn Reviewing bị trùng lặp khỏi triaging; công việc được hoàn thành khi hai trang không còn lặp lại cùng một thông tin.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Python
- Star
- 2.1k
- Fork
- 1k
- Merge trung bình
- 2 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 12
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của python/devguide
-
type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
topic-building python type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
needs: decision topic-test type-bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
topic-dev process type-feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Tất cả issue của python/devguide
Issue tương tự
-
货币战争手改优先级配置缺少列表元素类型校验（P3） Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
syfoud/Simulated_Scepter#172 ·
-
A cancelled tests run makes the coverage comment workflow fail and reports it as a red check on main Đang mởarea: ci bug perceived difficulty: 3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Nitjsefnie-Harness-Commons/daedalus#921 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
EleutherAI/lm-evaluation-harness#4207 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
ClickHouse/clickhouse-connect#1057 ·