IslasGECI / seleccion_analista_2022_gog

🐍 Guillermo Olvera Guerrero
GNU Affero General Public License v3.0
0 stars 0 forks source link

2️⃣📖🕵️‍♂️ LECTURA: The PR author’s guide to getting through code review #6

Closed devarops closed 2 years ago

devarops commented 2 years ago

¿Por favor podrías leer esta nota?

Te pido por favor que me digas:

MemoOlv commented 2 years ago

¿Qué parte me gustó más? -> Me gustó el tip de describir en tiempo imperativo la primer línea, lo pondré en práctica. Además de tratar de ser más descriptivo en el cuerpo del commit. Me gustaron mucho los ejemplos, son muy útiles.

¿Qué parte me gustó menos? -> Creo que se complementa mucho con el artículo de #3 . Creo que es más concreto pero con menos ilustraciones :(

¿Cuál parte resultó más clara? -> Creo que me queda más claro cómo separar la primera línea del cuerpo. Me parece que yo dejo mis commits hasta la primer línea

¿En cuál parte tuviste más dudas? -> Creo que no e quedó cuando es permitido hacer un CL largo. Entiendo que se tienen que tratar de mantener cortos y auto-contenidos, pero no entiendo el ejemplo que ponen. Supongo que es algo de práctica.

devarops commented 2 years ago

Hacer PR cortos es difícil. Sin duda con la práctica iremos adquiriendo esta habilidad.

¡Muchas gracias por realizar esta lectura!