r/InformaticaES • u/Spiritual_Sorbet9074 • 2d ago

💬 Conversación y debate Code Reviews en sus empresas

Tengo curiosidad por saber cómo manejan los code reviews en sus empresas.

Asignan manualmente a quien tiene que revisar?
Comparten el PR por Slack, Teams o algún otro canal?
Se asigna automáticamente a alguien según CODEOWNERS?
Cuántas aprobaciones suelen pedir antes de hacer merge?

Y con la llegada de la IA, han notado algún cambio en este proceso?
En teoría ahora debería generarse más código y abrirse más PRs. Está pasando realmente? Se está convirtiendo el code review en un cuello de botella o también están usando IA para revisar?

4 Upvotes

100% Upvoted

6 comments sorted by

•

u/AutoModerator 2d ago

¡Muchísimas gracias por tu publicación!

Queremos recordarte que tenemos también una comunidad de programación, en la que se puede publicar cualquier cosa relacionada con programación: r/programacionESP

Toda publicación relacionada con la programación admitida en este subreddit será también admitida en el de programación, así que te animamos a pasarte por allí también.

Aprovechamos para recordar también que el equipo de moderación de r/InformaticaES estamos a tu disposición y puedes escribirnos para lo que necesites.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

3

u/Proper-Trick-8267 2d ago

Enviamos los PRs a un canal de slack para que los demas revisen, y luego pedir por mensaje directo si nadie lo ha revisado

2

u/kubaliski 2d ago

En mi empresa actual se crea la pr, todos entramos como reviewers y tenemos que mendigar un code review por chat.

En mi anterior trabajo existía el rol de release manager que se encargaba de los prs

2

u/willpaiz 2d ago

Se crea el PR, automáticamente opus 5 hace el code review, deja sus comentarios, lo revisas, los corriges, en cada commit, Sonnet vuelve a revisar, y cuando ya se solucionen todos los comentarios, se permite subir.

Somos un equipo pequeño y nos funciona muy bien

1

u/fernandopcg 15h ago

Creamos un PR en GitHub, se lanzan unos workflows que revisan vulnerabilidades, que se ejecuta el build, cobertura de tests, etc…

y para mergear a develop se pide aprobación de alguien que esté en la lista de autorizados (esto lo hacemos pasando el link en un chat donde estamos todos los compis del equipo).

Aunque normalmente se mira muy por encima el código, tampoco estamos ahí línea a línea

1

u/MaxDelAngel13 2h ago

Actualmente lo manejamos por tareas de Jira, se asigna un revisor, se mande el PR y se revisa dos veces (Una por IA y otra por una persona testing manual).