git - code - Gerrit: combina múltiples compromisos en un solo "cambio"

jenkins

Question

Como práctica recomendada de Git, uno debería comprometerse con frecuencia, pero para revisar el código es posible que deba revisar un parche que consiste en múltiples confirmaciones a la vez. ¿Hay alguna manera de que se puedan revisar y confirmar las confirmaciones múltiples a la vez?

Answer 1

No, Gerrit no admite actualmente los envíos por lotes en una sola revisión. Sin embargo, hay un par de otras opciones.

En $ DAYJOB, mi equipo usa ramas de características para cambios más grandes. Las confirmaciones más pequeñas se revisan / fusionan en la rama de características individualmente, pero la rama de características solo se fusiona una vez que todo está en un buen lugar y todos los desarrolladores están contentos.

Gerrit también admite ramas temáticas, que son una forma conveniente de agrupar confirmaciones relacionadas. Se discuten brevemente en la documentation . Estas confirmaciones aún deben revisarse / fusionarse individualmente, pero pueden agruparse rápidamente en la interfaz de usuario web.

Answer 2

Si necesita actualizar las solicitudes de revisión ya publicadas, puede aprovechar las enmiendas de confirmación:

git commit --amend -C HEAD

y luego presionar para la revisión consecuente.

Creo que las confirmaciones públicas deben ser atómicas y contener el conjunto completo de funcionalidades a las que desea contribuir. Por lo general, no desea compartir todas sus confirmaciones intermedias. Así que el squashing se compromete antes de la revisión es una buena idea.

Answer 3

Una cosa es hacer una fusión de squash en una rama temporal y luego publicar ese cambio para revisión.

git checkout -b feature
git commit -m "start feature"
...
git commit -m "finish feature"
git checkout -b feature-review master
git merge --squash feature
git commit

Ahora su rama de feature-review contendrá la misma diferencia con respecto al master que la feature , pero con solo una confirmación.