Que tan grave es hacer un pull request con mucho commits en GitHub?

[PosicionamientoSeo]

Hola,

Trabajando en la web de un cliente hice un pull request y ademas de mi commit tambien habia otros 8 commits de otros developers.

El cliente me remarco que haga un pull request por commit, se notaba bastante enojado.

Es tan grave el pull request que hice? Es considerado ese nivel de error?

[pzin]

Es más fácil de inspeccionar un commit que varios. Localmente es bueno tener varios, más reducidos, y crear "unidades lógicas de trabajo". Entonces ahí tienes un dilema. Creo que lo que suele hacerse en estos casos es usar rebase para transformar tus (varios) commits en uno y así luego lo mandas para hacer el pull request.

No sé si habrá otra forma. El caso es que hay que tener cuidado con git rebase porque sobreescribes el historial del repositorio/rama.

Todo esto es teoría que en su día hablé y leí con otra persona. Seguramente otro aporte algo más sustancial.

Lo que no entiendo muy bien es que hayan otros commits de otros desarrolladores en lo tuyo. Igual es normal, pero a mi me parece extraño. No sé...

PD: me gusta tu firma nostálgica.

[pateketrueke]

No es ningún pecado hacer pull-requests con más de un sólo commit, que claro si es más limpio para todos hacer rebase, pero me imagino que estará bien con menos de 10 commits por PR.

Además la idea es que los commits sean atómicos.

Lo que no entiendo muy bien es que hayan otros commits de otros desarrolladores en lo tuyo. Igual es normal, pero a mi me parece extraño. No sé...

Eso también me preocupa, no es normal que alguien más pueda agregar commits a un PR que abriste.