Extract shared query parameter building for issue list operations
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- github
- Lĩnh vực
- api
Hướng nghiên cứu
Compare the query parameter building in GetRepositoryIssues.apply() and Repository.get_issues(), including their recover blocks, conditionals, and push calls. Read nearby model convenience methods to assess whether a shared helper fits the broader pattern. Done means the duplicated logic is consolidated without changing issue-list query behavior.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The query parameter building logic in GetRepositoryIssues.apply() and Repository.get_issues() is identical — same recover block, same conditionals, same push calls. If a new parameter is added, both sites must be updated in lockstep.
Repository.get_issues() can't delegate to GetRepositoryIssues.apply() because it has a template URL from the API response rather than owner/repo strings. It could potentially delegate differently, or the shared query param building could be extracted into a helper.
This is a pre-existing pattern in the codebase (other convenience methods on models also duplicate param building), so this may be worth addressing more broadly rather than just for issues.
- Ngôn ngữ chính
- Pony
- Star
- 7
- Fork
- 2
- Merge trung bình
- 2 giờ 26 phút
- Pull request đã merge (30 ngày)
- 7
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 ponylang/github_rest_api
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
ponylang/github_rest_api#108 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
ponylang/github_rest_api#92 ·
Tất cả issue của ponylang/github_rest_api
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
microsoft/magentic-ui#588 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
alexgorbatchev/simple-ptt#3 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
CorrelAid/formtransform#44 ·