Pygithub: Agregue soporte para API de protección de sucursales avanzadas

@harlowja Hice un poco de esto en https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

Fue suficiente para mí al menos auditar las protecciones de nuestras sucursales. YMMV.

Sí, sería genial tener algo así fusionado (y lanzado); esta biblioteca parece estar rezagada con respecto a la API y/o capacidades reales, lo que me entristece un poco :-/

Especialmente me gustaría esto ya que he estado tratando de asegurarme de que toda mi configuración de github para mi grupo esté en formato yaml, y no esté oculta en la interfaz de usuario de github, y luego tenga un programa que analice dicho formato yaml y ajuste varios repositorios de github. Esto crea continuidad (y cordura) en los repositorios de mis grupos y tener la capacidad de ajustar mediante programación las protecciones de los revisores sería lógicamente algo que un equipo querría hacer (para que la lista de revisores se pueda controlar y administrar a través de un formato yaml similar).

¡Buen trabajo @a13m!
Mientras tanto: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

Parece que estamos atascados hasta que alguien reúna la energía para abordar esto.

ahora debe pasar el objeto required_pull_request_reviews al llamar al extremo de protección de rama de actualización. Pasar este objeto era opcional durante el período de vista previa.

Me encantaría ver esto también. PR por favor ❤️

De hecho, esto es un gran lío. Repository.protect_branch() llama a un punto final que probablemente se eliminó, ya que GitHub publicó en su blog que sería cuando se graduara Branch Protection. Me he estado preguntando por qué los documentos de la API y el código están completamente en desacuerdo entre sí.

Voy a agregar un nuevo método Repository.edit_branch_protection() para llamar al nuevo punto final con las nuevas estructuras de datos, y estoy tentado a eliminar Repository.protect_branch() al mismo tiempo por estar roto.

Hola amigos, mi parche aún está incompleto, pero pensé que sería bueno tener comentarios tempranos ^

También he estado trabajando en esto, acercándolo de una manera diferente: agregando los métodos a Branch y eliminando Repository.protect_branch()

@stevenk ah, lo veo ahora. Hmm no quiero duplicar el trabajo. Mi intención era mantener la API de la biblioteca igual. ¿Cómo quieres hacerlo?

@JPWKU Lo siento, debería haber comunicado más claramente que estaba trabajando activamente en esto. El método Repository.protect_branch() coincidía con la API de GitHub, hasta que lo eliminaron cuando se graduó la nueva API de protección de sucursales.

Mi parche actual implementa la API de protección de sucursales como muestra GitHub en https://developer.github.com/v3/repos/branches/ , como consecuencia, es bastante grande. Tengo más o menos 3 casos de prueba para terminar de escribir antes de impulsar esto.

@stevenk no se preocupe, dado que ya tiene trabajo en esta área, cerraré mi PR y comentaré / ayudaré con el suyo.

@stevenk ¿Planea fusionar esto pronto? Necesito esta funcionalidad.

@prburgu No depende de mí cuando se fusiona. @JPWKU , ¿cuándo tiene tiempo para continuar revisando el PR adjunto?

@prburgu @stevenk Voy a tratar de sacar tiempo hoy o esta noche para terminar la reseña.

¿Algún avance en esto?