Merge request

Merge request võimaldab ühendada koodi erinevatest harudest ühesse. Mergimise oskus on üks kõige olulistematest terve IT-sfääris.

Loomine

Kui feature branchil olev kood on koodi ülevaataja ja testija hinnangul korras, võib selle maini mergida. Esiteks tuleb luua merge request. Selleks otsi üles plussmärk ning vali “New merge request”. Vali source branchiks dev ning vajuta “Compare branches and continue”.

../../_images/merge_create1.gif

Merge-imine

Nüüd tuleb üle vaadata, kas kood on mergimiseks valmis ning seejärel valida “Merge”.

../../_images/mergimine1.gif

Oluline koodi ülevaatajale

  • Loe ja proovi aru saada, mis on merge request-i eesmärk, mida ta muudab

  • Klooni endale haru, et testida uusi lisandusi

  • Veendu, et kood oleks selgesti mõistetav (muutujate/meetodite nimed arusaadavad, kommentaarid keerulisemate meetodite jaoks, docstringid, jne)

  • Veendu, et stiil oleks korras (pole liiga lohisevad meetodid, nimed õigete algustähtedega, jne)

  • Testi põhjalikult - proovi leida olukordi, mida kood ei kata või mille puhul rakenduse töö on häiritud. Kontrolli, et ka teised seotud funktsionaalsused töötavad ning uus kood pole vana katki teinud.

  • Kui merge request-iga on seotud pilet, siis kontrolli, et kõik piletis olevad kriteeriumid on täidetud

  • Anna konstruktiivset tagasisidet - lisa kommentaare; mis on hästi/mida parandada; kui võimalik, siis ütle võimalikult täpselt millised klassid/failid/read on vigased. GitLabis (ja ka paljudes teistes projektihaldustarkvarades) saab seda teha merge requesti korral täpse rea juures.

Video