Mantengo un repositorio de github relativamente popular.
Cuando una solicitud de extracción es buena para fusionar, generalmente le pido al autor que la cambie a una sola confirmación antes de fusionarla (especialmente cuando ha habido varias ediciones pequeñas).
¿Es esta una buena práctica de git? ¿Es esta la etiqueta GitHub aceptable / estándar?
Entonces algunos beneficios:
- Me sale un buen historial de confirmación limpio en los registros de confirmación
- No necesito alterar el compromiso yo mismo
- Delega parte del trabajo.
Algunos posibles inconvenientes:
- No estoy seguro si esto es buena etiqueta
- No estoy seguro de si esta es una buena práctica de git
- Por lo general, ya he pedido algunos otros cambios, este es uno más y no quiero desanimar a los contribuyentes.
git push -f
?Respuestas:
En lo que respecta a Git, es algo así como una guerra santa si simplemente debes fusionar ramas o rebasar los commits en la última versión de la rama en la que te estás fusionando. Hay muchas conversaciones sobre cuál es mejor si haces una búsqueda rápida en Programmers.SE .
En cuanto a la etiqueta detrás de esto, tratemos esto desde una perspectiva práctica. Cuando se trata de código nuevo que viene de otra persona, siempre es mejor hacer que fusionen los últimos cambios de la rama o que lo vuelvan a crear antes de fusionarlo para garantizar una fusión limpia. Recuerde, escribieron el código, por lo que generalmente son, con mucho, los más calificados para lidiar con cualquier conflicto de fusión / rebase. Personalmente no veo ningún problema con eso, y veo esta solicitud todo el tiempo de otras personas. Para mí, si no hay conflictos, a menudo lo haré yo mismo, ya que es una actualización de dos segundos que git puede aplicarse. Pero si hay conflictos, siempre le pediré al autor original del código que se encargue de ello.
Además, para GitHub (como mínimo) específicamente, mostrarán un enlace a su
CONTRIBUTING
archivo sobre cualquier intento de relaciones públicas, por lo que es un buen lugar para delinear sus expectativas y muchos proyectos incluyen que solo fusionarán ramas actualizadas.fuente