Pygithub: Fügen Sie Unterstützung für APIs für erweiterten Zweigschutz hinzu

@harlowja Ich habe ein bisschen davon in https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca gemacht

Mir hat es gereicht, zumindest unseren Filialschutz zu auditieren. YMMV.

Ja, es wäre supernett, so etwas einzufügen (und freizugeben); diese Bibliothek scheint hinter der eigentlichen API und/oder den Fähigkeiten hinterherzuhinken, was mich etwas traurig macht :-/

Das würde mir besonders gefallen, da ich versucht habe, sicherzustellen, dass meine gesamte Github-Konfiguration für meine Gruppe in einem Yaml-Format vorliegt und nicht in der Github-Benutzeroberfläche versteckt ist, und dann ein Programm habe, das das Yaml-Format analysiert und verschiedene Github-Repositories anpasst. Dies schafft Kontinuität (und Vernunft) in den Repositorys meiner Gruppen, und die Möglichkeit, den Prüferschutz programmgesteuert anzupassen, wäre logischerweise etwas, das ein Team tun möchte (damit die Prüferliste über ein ähnliches Yaml-Format gesteuert und verwaltet werden kann).

Gute Arbeit @a13m!
Inzwischen: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

Sieht so aus, als stecken wir fest, bis jemand die Energie aufbringt, dies anzugehen.

Sie müssen jetzt das Objekt required_pull_request_reviews übergeben, wenn Sie den Update-Branch-Protection-Endpunkt aufrufen. Das Übergeben dieses Objekts war während des Vorschauzeitraums optional.

Das würde ich auch gerne sehen. PR bitte ❤️

Das ist in der Tat ein großes Durcheinander. Repository.protect_branch() ruft einen Endpunkt auf, der wahrscheinlich entfernt wurde, da GitHub gebloggt hat, dass dies der Fall sein würde, wenn Branch Protection abgeschlossen wäre. Ich habe mich gefragt, warum die API-Dokumentation und der Code völlig im Widerspruch zueinander stehen.

Ich werde eine neue Methode Repository.edit_branch_protection() hinzufügen, um den neuen Endpunkt mit den neuen Datenstrukturen aufzurufen, und ich bin versucht, gleichzeitig Repository.protect_branch() zu entfernen, weil es kaputt ist.

Hallo Leute, mein Patch ist noch unvollständig, aber ich dachte, es wäre schön, bald Feedback zu bekommen ^

Ich habe auch daran gearbeitet und es auf eine andere Art und Weise angegangen – indem ich die Methoden zu Branch hinzugefügt und Repository.protect_branch() gelöscht habe.

@stevenk ah das sehe ich jetzt. Hmm, ich will keine Doppelarbeit machen. Meine Absicht war es, die Bibliotheks-API gleich zu halten. Wie wollen Sie vorgehen?

@JPWKU Es tut mir leid, ich hätte klarer kommunizieren sollen, dass ich aktiv daran arbeite. Die Methode Repository.protect_branch() stimmte mit der GitHub-API überein, bis sie entfernt wurde, als die neue Zweigschutz-API abgeschlossen wurde.

Mein aktueller Patch implementiert die Zweigschutz-API, wie GitHub auf https://developer.github.com/v3/repos/branches/ zeigt, daher ist er ziemlich groß. Ich muss ungefähr 3 Testfälle fertig schreiben, bevor ich das hochschiebe.

@stevenk keine Sorge, da Sie bereits Arbeit in diesem Bereich haben, werde ich meine PR schließen und Ihre kommentieren / helfen.

@stevenk Planen Sie, dies bald zusammenzuführen? Ich brauche diese Funktion.

@prburgu Es liegt nicht an mir, wann es zusammengeführt wird. @JPWKU Wann haben Sie etwas Zeit, um die beigefügte PR weiter zu überprüfen?

@prburgu @stevenk Ich werde versuchen, mir heute oder heute Abend Zeit zu nehmen, um die Rezension fertigzustellen.

Gibt es hierzu Neuigkeiten?