Pygithub: Adicionar suporte para APIs avançadas de proteção de ramificação

Criado em 30 jun. 2017  ·  15Comentários  ·  Fonte: PyGithub/PyGithub

Parece que as seguintes APIs existem no github v3 (como beta?) mas ainda não estão refletidas nesta biblioteca.

Seria ótimo ter suporte total a essa API (especialmente porque permite controlar as revisões de PR e quantas são necessárias e por quem).

feature request high priority

Comentários muito úteis

Esta é de fato uma grande confusão. Repository.protect_branch() chama em um endpoint que provavelmente foi removido, já que o GitHub postou no blog que seria quando o Branch Protection se graduasse. Eu tenho me perguntado por que os documentos da API e o código estão completamente em desacordo um com o outro.

Vou adicionar um novo método Repository.edit_branch_protection() para chamar o novo endpoint com as novas estruturas de dados, e estou tentado a remover Repository.protect_branch() ao mesmo tempo por estar quebrado.

Todos 15 comentários

@harlowja eu fiz um pouco disso em https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

Foi o suficiente para eu pelo menos auditar nossas proteções de filiais. YMMV.

Sim, seria superlegal ter algo assim integrado (e lançado); esta biblioteca parece estar atrasada em relação à API e/ou recursos reais, o que me deixa um pouco triste :-/

Especialmente gostaria disso, pois estou tentando garantir que toda a minha configuração do github para o meu grupo esteja em um formato yaml e não escondida na interface do usuário do github e, em seguida, tenha um programa que analisa o formato yaml e ajusta vários repositórios do github. Isso cria continuidade (e sanidade) nos repositórios dos meus grupos e ter a capacidade de ajustar programaticamente as proteções do revisor seria logicamente algo que uma equipe gostaria de fazer (para que a lista de revisores possa ser controlada e gerenciada por meio de um formato yaml semelhante).

Bom trabalho @a13m!
Enquanto isso: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

Parece que estamos presos até que alguém reúna energia para lidar com isso.

agora você precisa passar o objeto required_pull_request_reviews ao chamar o endpoint de proteção de branch de atualização. A passagem deste objeto era opcional durante o período de visualização.

Eu adoraria ver isso também. PR por favor ❤️

Esta é de fato uma grande confusão. Repository.protect_branch() chama em um endpoint que provavelmente foi removido, já que o GitHub postou no blog que seria quando o Branch Protection se graduasse. Eu tenho me perguntado por que os documentos da API e o código estão completamente em desacordo um com o outro.

Vou adicionar um novo método Repository.edit_branch_protection() para chamar o novo endpoint com as novas estruturas de dados, e estou tentado a remover Repository.protect_branch() ao mesmo tempo por estar quebrado.

Oi pessoal, Meu patch ainda está incompleto, mas achei que seria bom ter um feedback antecipado ^

Eu também tenho trabalhado nisso, abordando de uma maneira diferente -- adicionando os métodos ao Branch e descartando Repository.protect_branch()

@stevenk ah eu vejo isso agora. Hmm não quero duplicar o trabalho. Minha intenção era manter a biblioteca api da mesma forma. Como você quer fazer isso?

@JPWKU Desculpe, eu deveria ter comunicado mais claramente que estava trabalhando ativamente nisso. O método Repository.protect_branch() correspondia à API do GitHub, até que foi removido quando a nova API de proteção de ramificação foi graduada.

Meu patch atual implementa a API de proteção de branch como o GitHub mostra em https://developer.github.com/v3/repos/branches/ , como consequência, é bastante grande. Eu tenho 3 ou mais casos de teste para terminar de escrever antes de empurrar isso.

@stevenk não se preocupe, já que você já tem trabalho nessa área vou fechar meu PR e comentar/ajudar o seu.

@stevenk Você planeja mesclar isso em breve? Estou precisando dessa funcionalidade.

@prburgu Não depende de mim quando se funde. @JPWKU quando você tem algum tempo para continuar revisando o PR anexado?

@prburgu @stevenk Vou tentar arranjar tempo hoje ou esta noite para terminar a revisão.

alguma atualização disso?

Esta página foi útil?
0 / 5 - 0 avaliações