Mi opinión es que se debe hacer una revisión del código en cada cambio. Si corrige un error, haga que alguien haga una revisión rápida del código. ¿Escribes una nueva característica? Revisión de código. ¿Refactorizas algunas cosas? Revisión de código.
Más específicamente, asegúrese de que cada CR tenga exactamente una unidad de trabajo. Por ejemplo, solucionó un error o agregó una nueva característica (o una sola pieza de esa característica). No combine múltiples tareas en un solo CR, ya que esto puede hinchar el CR y confundir a sus compañeros, y puede hacer que sea más probable que se pierda algo importante. Al limitar sus CR a una sola unidad, enfoca al revisor en ese cambio específico. Esto ayuda mucho.
Tenga en cuenta que esto no significa solo un archivo por CR. Podrías (y a menudo lo harás) tocar muchas cosas para realizar esa unidad de trabajo. Pero al limitar la revisión a solo ese cambio, usted afirma que todo lo que está en la revisión está relacionado y es relevante, y que hay cambios mínimos o ningún cambio que no sea relevante.
- ¿Cuáles son los salarios para los ingenieros de software al principio?
- ¿Existe un servicio web que le diga cuánto tardan en cargarse las páginas de su sitio en varios países del mundo?
- ¿Qué es la prueba Big Bang en ingeniería de software?
- ¿Cuáles son los desafíos de ingeniería en ZenPayroll?
- ¿Cuáles son los diferentes tipos de herramientas para desarrolladores?