Move compatibility for non github.com targets to `github.With<THING>` options
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- go
- Lĩnh vực
- api, backend-api-design
Hướng nghiên cứu
Bắt đầu bằng cách xem xét thiết lập client mặc định, tùy chọn github.WithEnterpriseURLs hiện có và các bước kiểm tra khả năng tương thích được mô tả trong đề xuất. Theo dõi cách các phiên bản API và các đích không phải github.com hiện được cấu hình. Được xem là hoàn tất khi hành vi được đề xuất của WithAPIVersion, WithAdvancedServer và WithEnterpriseCloud được thiết kế và triển khai nhất quán, đồng thời tùy chọn đã lỗi thời được xử lý.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I propose that all compatibility functionality not required for the github.com target be removed from the default client setup and only be included via a github.With<THING> option. This (hopefully) would make the default client surface area simpler to understand. The justification for this is that you already need to use github.WithEnterpriseURLs to target non github.com. I propose the following changes to adopt this pattern, with the potential for other capabilities (such as enabling/disabling preview endpoints) to be added later.
- Default to the latest API version (
2026-03-10)- Functions that require a specific version now check against the client version and error if not valid
- Add
github.WithAPIVersionto explicitly set a version- Set at the client/service level
- Add
github.WithAdvancedServerto enable GHAS support- Better validation of inputs
- Support setting version
- Add compatibility accordingly
- Add
github.WithEnterpriseCloudto enable GHEC support- Better validation of inputs
- Deprecate
github.WithEnterpriseURLs
I'd appreciate any feedback on this proposal.
cc: @gmlewis - @alexandear - @zyfy29 - @Not-Dhananjay-Mishra - @munlicode
- Ngôn ngữ chính
- Go
- Star
- 11.3k
- Fork
- 2.5k
- Merge trung bình
- 2 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 43
Chuẩn bị môi trường
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 google/go-github
-
http.Client.Timeout is ignored by redirect-aware requestsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Add Copilot content exclusion set/get enterprise and org set endpointsCó thể đã có người làm @Tens1des đã nhận 30 ngày trước. Đang mở
google/go-github#4526 · 1 bình luận · 1 reaction · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add Copilot custom agents enterprise endpointsCó thể đã có người làm @Tens1des đã nhận 30 ngày trước. Đang mở
google/go-github#4524 · 1 reaction · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Repository ruleset rule type `code_quality` is not modelledCó thể đã có người làm @etifontaine đã nhận 33 ngày trước. Đang mở
google/go-github#4520 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Including an enterprise team as a reviewer for an environment results an unmarshal errorCó thể đã có người làm @JWilkinsonMB đã nhận 48 ngày trước. Đang mở
google/go-github#4457 · 1 reaction · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của google/go-github
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
war-and-code/dircue#200 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
actor/human kind/bug priority/important-soon triage-accepted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
kelos-dev/kelos#1804 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
`date` → `date-time` (and `time` → `date-time`) is classified as a widening, but a date is not a valid date-timeCó thể đã có người làm @reuvenharrison đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
fix: invalid GPU spec in SparkApplication is silently ignoredCó thể đã có người làm @pratik-naik003 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
kubeflow/spark-operator#3223 ·
Maintainer thường phản hồi trong vòng 5 ngày