Logo de Dallonses

Code review

Què és el code review?

El code review és el pas en què un altre enginyer llegeix un canvi abans que es fusioni al codi. Comprova si fa el que diu fer, si encaixa amb la resta del sistema i si algú serà capaç d'entendre'l d'aquí a sis mesos. Després l'aprova o demana canvis.

La majoria dels equips fan això a través d'un pull request. L'autor proposa un canvi, un revisor comenta, tots dos van i venen, i el canvi es fusiona un cop aguanta. Les comprovacions automàtiques corren en paral·lel: linters, type checks i la suite de proves atrapen els problemes mecànics perquè l'humà es pugui centrar en els de criteri. Un revisor detecta que una nova consulta a base de dades no té límit i escanejaria una taula que creix cada dia. Les proves van passar. El codi funcionava. Tot i així hauria tombat producció sota trànsit real, i això és exactament el tipus de cosa que la revisió atrapa i l'automatització es perd.

Ben fet, difon coneixement tant com atrapa bugs. Ara dues persones entenen el canvi en lloc d'una. Mal fet, es torna un segell de goma o un lloc per a quisquilles. La diferència és si l'equip ho tracta com a propietat compartida de la qualitat.

Code review a Dallonses

Res no arriba a producció a Dallonses sense que un altre enginyer ho llegeixi. No és una formalitat que esmentem per sonar rigorosos. És com la feina es manté bona quan l'equip va ràpid i les persones canvien. La revisió és on s'ensenyen els estàndards, no en un document que ningú obre.

Mantenim les revisions centrades en el que importa. Correcció, claredat i si el canvi fa el sistema més fàcil o més difícil de mantenir. Les comprovacions mecàniques corren automàtiques als nostres pipelines CI/CD, així els revisors gasten la seva atenció en les decisions que una màquina no pot prendre. Quan treballem al costat dels enginyers d'un client, les nostres revisions es tornen una manera de compartir com pensem, i les seves afinen les nostres. Aquest intercanvi és part de per què els equips surten d'un projecte més forts del que hi van entrar.

Vols un segon parell d'ulls sènior sobre el teu codi? Parlem-ne.

Parlem de qualitat

Serveis relacionats


Preparat per a traballar junts?

Reserva una reunió
Aymón sostenint una revista Tools davant de la seva cara
Ari treballant en un portàtil a l'aire lliure envoltada de plantes
Vista superior d'un escriptori de fusta amb teclat, ratolí i auriculars
Il·lustració dibuixada a mà d'una mà chasquejant els dits
Nico recolzat contra un dispensador d'aigua al costat d'un extintor
Primer pla d'un ordinador obert amb placa de circuit i components sobre un escriptori de fusta
Bernat i Andreu col·laborant en un escriptori amb monitors i un portàtil
Il·lustració dibuixada a mà d'una mà oberta saludant
Aymón sostenint una revista Tools davant de la seva cara
Ari treballant en un portàtil a l'aire lliure envoltada de plantes
Vista superior d'un escriptori de fusta amb teclat, ratolí i auriculars
Il·lustració dibuixada a mà d'una mà chasquejant els dits
Nico recolzat contra un dispensador d'aigua al costat d'un extintor
Primer pla d'un ordinador obert amb placa de circuit i components sobre un escriptori de fusta
Bernat i Andreu col·laborant en un escriptori amb monitors i un portàtil
Il·lustració dibuixada a mà d'una mà oberta saludant