Eu mantenho um repositório github relativamente popular.
Quando uma solicitação pull é boa para mesclar, geralmente peço ao autor que a refaça novamente para uma única confirmação antes de mesclá-la (especialmente quando houver várias pequenas edições).
Essa é uma boa prática de git? Essa etiqueta é aceitável / padrão do GitHub?
Então, alguns benefícios:
- Recebo um bom histórico de confirmação limpa nos logs de confirmação
- Eu não preciso alterar o commit sozinho
- Delega parte do trabalho
Algumas desvantagens possíveis:
- Não tenho certeza se isso é uma boa etiqueta
- Não tenho certeza se essa é uma boa prática de git
- Normalmente, já pedi algumas outras alterações - essa é mais uma e não quero desencorajar colaboradores.
git push -f
?Respostas:
No que diz respeito ao Git, é uma espécie de guerra santa se você deve simplesmente mesclar ramificações ou refazer as confirmações na versão mais recente da ramificação na qual está se fundindo. Há muitas conversas sobre o que é melhor se você fizer uma pesquisa rápida no Programmers.SE .
Quanto à etiqueta por trás disso, vamos lidar com isso de uma perspectiva prática. Ao lidar com o novo código vindo de outra pessoa, é sempre melhor fazer com que ela mescle as alterações mais recentes da ramificação ou refazê-las novamente antes de mesclar para garantir uma mesclagem limpa. Lembre-se de que eles escreveram o código e, portanto, são de longe os mais qualificados para lidar com qualquer conflito de mesclagem / rebase. Pessoalmente, não vejo problemas com isso e vejo esse pedido o tempo todo de outras pessoas. Para mim, se não houver conflitos, geralmente eu mesmo faço isso já que é uma atualização de dois segundos que o git pode se aplicar. Mas se houver conflitos, sempre solicitarei ao autor original do código que lide com eles mesmos.
Além disso, especificamente para o GitHub (no mínimo), eles exibirão um link para seu
CONTRIBUTING
arquivo acima de qualquer tentativa de relações públicas, de modo que seja um bom local para delinear suas expectativas e muitos projetos incluem que eles apenas se unirão às ramificações atualizadas.fonte