Kā tika organizēta jūsu komandas kodu pārskatīšanas procedūra?

Mēs izmantojām kopīgu izstrādi, izmantojot versiju kontroli sistēmu Git un platformu repozitoriju pārvaldībai (piemēram, Bitbucket vai GitLab). Koda pārskatīšanas process tika organizēts šādi:

1. Zara izveide: Izstrādātājs izveido atsevišķu zaru jaunas funkcijas vai kļūdas labojuma veikšanai (git checkout -b feature/my-new-feature).
2. Izstrāde un komiti: Zārā tiek veiktas izmaiņas, kas tiek reģistrētas kā komiti (git add ., git commit -m "Initial feature implementation").
3. Poga uz attālo repozitoriju: Zars tiek nosūtīts uz attālo repozitoriju (git push origin feature/my-new-feature).
4. Pull/Merge pieprasījuma izveide: Izstrādātājs izveido Pull Request (Bitbucket) vai Merge Request (GitLab), norādot mērķa zaru (piemēram, develop vai main). Aprakstā ir sniegts īss izmaiņu apraksts, uzdevumu saites uz izsekošanas sistēmu (Jira, Trello utt.) un pievienotie faili (ekrānuzņēmumi, video, diagrammas).
5. Recenzentu piešķiršana: Piešķir vienu vai divus izstrādātājus pārskatīšanai.
6. Pārskatīšanas process: Recenzenti pārskata izmaiņas Pull/Merge pieprasījumā. Viņi var atstāt komentārus, ieteikt uzlabojumus, uzdot jautājumus:

   // Piemērs komentāram pārskatīšanas laikā
   function fetchData() {
     // Varbūt būtu jāpiekrīt kļūdu apstrādei datu pieprasījumā
     return fetch('/api/data');
   }
   

7. Izmaiņu ieviešana, balstoties uz pārskatīšanas rezultātiem: Izstrādātājs veic nepieciešamās izmaiņas savā zarā un izveido jaunus komitus.
8. Pull/Merge pieprasījuma atjaunināšana: Izmaiņas automātiski parādās Pull/Merge pieprasījumā pēc to nosūtīšanas uz attālo repozitoriju.
9. Atkārtota pārskatīšana: Recenzenti vēlreiz pārskata izmaiņas un dod apstiprinājumu.
10. Zara apvienošana: Pēc apstiprinājuma zars tiek apvienots ar mērķa zaru (parasti automātiski platformas līdzekļiem pēc CI/CD piltuves iziešanas).

Mēs arī izmantojām CI/CD piltuvi, lai automātiski palaistu vienību testus, statisko koda analīzi (ESLint, Prettier) un projekta būvi pirms apvienošanas, kas palīdzēja agrīni atklāt kļūdas.

Veiksmīgas pārskatīšanas kritēriji ietvēra:

• Atbilstība uzdevuma prasībām.
• Komandas koda stila ievērošana.
• Kritiskās loģikas vienību testu esamība.
• Acīmredzamu kļūdu un "kostīļu" neesamība.
• Koda saprotamība citiem komandas locekļiem.