Cómo mejorar la revisión de las solicitudes de retiro (sin soluciones de gobernanza) En la tecnología, siempre estamos trabajando para escribir un código mejor, más fuerte y más sostenible. Las revisiones de código son una parte esencial de ese proceso. Sin embargo, es una habilidad que rara vez nos enfocamos en mejorar. Convertirse en un revisor de calidad puede convertirte en un verdadero activo para tu equipo, pero no es fácil.Muchos de nosotros abordamos las revisiones de código desde el ángulo equivocado o con fines equivocados. En la Puede transformar la dinámica de su equipo y ponerle en un camino hacia el éxito. Las cosas correctas El camino correcto The Purpose of a PR Review El propósito de una revisión de PR Como ingenieros, nuestro trabajo principal es escribir código. Para resolver problemas. Para mantener el mundo girando un bit o byte a la vez. Pero ¿cuál es nuestro papel al revisar el código? esa pregunta puede ser molesta, especialmente la primera vez que se le pide revisar el código de otra persona. So, let’s start with what you’re not expected to do: No se espera que resuelva el problema para el autor. No se espera que reescriba su código para coincidir con su propia visión. So, what are you responsible for? El éxito de un equipo a menudo depende de la calidad del código que lo convierte en producción y de la rapidez con la que se descubren y resuelven los problemas críticos. El problema planteado ha sido resuelto, la solución es limpia y probable, y El código es eficiente y completo. A menudo, la solución se verá muy diferente de cómo lo habrías resuelto, y eso está bien.. Eso no es relevante para la revisión.Lo que importa es si el código funciona, es mantenible y puede ser probado de manera efectiva. Otro gran beneficio de las revisiones es la transferencia de conocimientos.Todo el mundo resuelve problemas de manera diferente, y la revisión del código es una oportunidad para que el equipo aprenda el uno del otro. Así que, subamos un poco más y hablemos de por qué estás haciendo esto. Is This Done? ¿Es esto hecho? Todos hemos estado allí: usted comienza a resolver un problema, descubre una idea tanxencial fresca, y de repente está construyendo algo completamente diferente. Usted llega al final y usted con entusiasmo mantiene su idea para que todos vean, sólo para darse cuenta de que no realmente resolvió el problema que se planteó para resolver. Usted construyó algo fresco y sorprendente, pero no resuelve el propósito previsto. La revisión de PR puede ayudarnos a evitar estas pequeñas equivocaciones de convertirlo en producción y causar problemas a los clientes. El revisor debe centrarse en el problema en la mano. Tener una pequeña familiaridad con los boletos asociados con este PR, haber leído el README o comentarios en el PR, y tener una sólida comprensión del problema que se está resolviendo. Queremos estar seguros de que el ingeniero también entendió el problema y abordó el problema correcto. Housekeeping Domiciliación La mayoría de nosotros escribiremos código de forma iterativa, lo que significa que solucionamos el problema instintivamente, pero luego reescribimos el código varias veces para que sea lo más limpio, claro y eficiente posible. Pero a veces, no hacemos tantas iteraciones como podríamos tener que llegar a la verdadera línea de fondo. Aquí está la parte difícil: tu trabajo no es juzgar si la solución es la más limpia posible, sino si es lo suficientemente limpia como para ser probable y mantenible. Esta es una línea muy fina que puede ser difícil de atravesar. Una revisión debe centrarse en si este código está en un estado en el que será fácil de probar y mantener; eso es lo que es.Leer el código y imaginar futuros errores o cambios que podrían surgir, y ¿cuán difícil será este código para corregir o cambiar?Si parece confuso o complicado, probablemente deberíamos simplificarlo. Sin embargo, si el código parece lo suficientemente fácil como para trabajar con, pero puede imaginar una solución aún mejor, está bien llamar eso, pero no deje que esto bloquee la revisión. Usted no está aquí para resolver el problema para el autor, sino para asegurarse de que ellos mismos han resuelto el problema. consejos y trucos nos ayudan a aprender y crecer, por lo que su consejo es bienvenido, pero no lo convierta en un problema de bloqueo. Conclusion Conclusión Las revisiones de código no se tratan de demostrar que eres el ingeniero más inteligente de la habitación o de forzar que cada solución coincida con tu estilo. Si se aproxima a las revisiones centrándose en El código realiza en lugar de Cuando llegue allí, ayudará a que su equipo se mueva más rápido, construya confianza y fomente la colaboración. Qué Cómo En el siguiente capítulo, entraremos en la : los principios y prácticas que hacen que una revisión sea efectiva, respetuosa y productiva.Porque incluso si conoces el propósito, se necesita una verdadera habilidad para entregar feedback que eleva tanto el código como el codificador. Cómo Publicado originalmente: https://substack.com/@halexmorph
