Move compatibility for non github.com targets to `github.With<THING>` options
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- go
- Bereich
- api, backend-api-design
Rechercherichtung
Beginne mit der Überprüfung der standardmäßigen Client-Einrichtung, der vorhandenen Option github.WithEnterpriseURLs und der im Vorschlag beschriebenen Kompatibilitätsprüfungen. Verfolge, wie API-Versionen und Ziele außerhalb von github.com derzeit konfiguriert werden. Als erledigt gilt die Aufgabe, wenn das vorgeschlagene Verhalten von WithAPIVersion, WithAdvancedServer und WithEnterpriseCloud konsistent entworfen und implementiert ist und die veraltete Option berücksichtigt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- Go
- Sterne
- 11.3k
- Forks
- 2.6k
- Ø Merge
- 2 T. 16 Std.
- Gemergte PRs (30 T.)
- 35
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus google/go-github
-
Add support for Actions cache retention and storage limits for repositories and enterprisesEvtl. vergeben @bhuvaneshkumar-cyber hat das vor 2 Tagen übernommen. Offen
google/go-github#4645 · 1 Reaktion · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 5/100
google/go-github#4636 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
http.Client.Timeout is ignored by redirect-aware requestsEvtl. vergeben @sb123sb123 hat das vor 29 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
Add Copilot content exclusion set/get enterprise and org set endpointsEvtl. vergeben @Tens1des hat das vor 43 Tagen übernommen. Offen
google/go-github#4526 · 1 Kommentar · 1 Reaktion · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Add Copilot custom agents enterprise endpointsEvtl. vergeben @Tens1des hat das vor 43 Tagen übernommen. Offen
google/go-github#4524 · 1 Reaktion · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in google/go-github
Ähnliche Issues
-
[BUG]: Windows GUI package of v1.17.0-rc3 has no GTK DLLsEvtl. vergeben @andrut-ds hat das heute übernommen. Offenbug CI/CD Pactus-GUI Windows
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
pactus-project/pactus#2428 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 91/100
Maintainer antworten meist innerhalb von 1 Tag
-
feat
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
ory/hydra-maester#312 · 1 Reaktion ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
rossoctl/context-guru#610 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
Maintainer antworten meist innerhalb von 1 Tag