Pygithub: Ajout de la prise en charge des API de protection de branche avancées

@harlowja j'ai fait un peu de cela dans https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

Il me suffisait d'au moins auditer les protections de nos succursales. YMMV.

Ya, ce serait super bien d'avoir quelque chose comme ça fusionné (et publié); cette bibliothèque semble être en retard par rapport à l'API et/ou aux capacités réelles, ce qui me rend un peu triste :-/

J'aimerais particulièrement cela depuis que j'ai essayé de m'assurer que toute ma configuration github pour mon groupe est au format yaml, et non cachée dans l'interface utilisateur github, puis d'avoir un programme qui analyse ledit format yaml et ajuste divers référentiels github. Cela crée une continuité (et un bon sens) dans les référentiels de mes groupes et avoir la possibilité d'ajuster par programmation les protections des réviseurs serait logiquement quelque chose qu'une équipe voudrait faire (afin que la liste des réviseurs puisse être contrôlée et gérée via un format yaml similaire).

Beau travail @a13m !
En attendant : https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

On dirait qu'on est coincés jusqu'à ce que quelqu'un rassemble l'énergie pour s'attaquer à ça.

vous devez désormais transmettre l'objet required_pull_request_reviews lors de l'appel du point de terminaison de protection de la branche de mise à jour. Passer cet objet était facultatif pendant la période de prévisualisation.

J'aimerais bien voir ça aussi. PR s'il vous plaît ❤️

C'est en fait un grand gâchis. Repository.protect_branch() appelle un point de terminaison qui a probablement été supprimé, puisque GitHub a annoncé sur le blog que ce serait le cas lorsque Branch Protection obtiendrait son diplôme. Je me demandais pourquoi la documentation de l'API et le code étaient complètement en contradiction l'un avec l'autre.

Je vais ajouter une nouvelle méthode Repository.edit_branch_protection() pour appeler le nouveau point de terminaison avec les nouvelles structures de données, et je suis tenté de supprimer Repository.protect_branch() en même temps pour être cassé.

Salut les gens, Mon patch est encore incomplet mais j'ai pensé que ce serait bien d'avoir un retour rapide ^

J'ai également travaillé là-dessus, en l'abordant d'une manière différente - en ajoutant les méthodes à Branch et en supprimant Repository.protect_branch()

@stevenk ah je vois ça maintenant. Hmm ne veux pas dupliquer le travail. Mon intention était de garder la même API de bibliothèque. Comment voulez-vous vous y prendre ?

@JPWKU Je suis désolé, j'aurais dû communiquer plus clairement que je travaillais activement là-dessus. La méthode Repository.protect_branch() correspondait à l'API GitHub, jusqu'à ce qu'elle soit supprimée lorsque la nouvelle API de protection de branche est graduée.

Mon correctif actuel implémente l'API de protection des branches comme le montre GitHub sur https://developer.github.com/v3/repos/branches/ , par conséquent, il est plutôt volumineux. J'ai environ 3 cas de test à terminer d'écrire avant de pousser cela.

@stevenk pas de soucis, puisque vous avez déjà du travail dans ce domaine, je vais fermer mon PR et commenter/aider le vôtre.

@stevenk Prévoyez-vous de fusionner cela bientôt ? J'ai besoin de cette fonctionnalité.

@prburgu Ce n'est pas à moi de décider quand ça fusionne. @JPWKU , quand avez-vous le temps de continuer à examiner le PR ci-joint ?

@prburgu @stevenk Je vais essayer de trouver du temps aujourd'hui ou ce soir pour terminer l'examen.

Une mise à jour pour ceci?