Merge request

Pärast töö lõpetamist teatud harul või hargnemisel tuleks arendajal esitada merge request, mis sisaldab tehtud muudatusi, eesmärgiga ü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.

  1. Selleks otsi üles plussmärk ning vali “New merge request”

  2. Vali source branchiks dev

  3. Vajuta “Compare branches and continue”

Tegelikult on hea ka lisada reviewer, ehk keegi teine tiimist.

  1. Vajuta alale "Unassigned" reviewer'i juures

  2. Kirjuta sinna mõne teise tiimiliikme uni-id ning vali nemad ülevaatajaks

../../_images/merge_create1.gif

Merge-imine

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

../../_images/mergimine1.gif

Merge request-i ülevaatamine ja aksepteerimine

Merge requesti peaks alati vaatama üle keegi teine meeskonnaliikmetest, põhjusel, et esitajale endale võivad mõned asjad kahe silma vahele jääda.

Merge request tuleks vaadata hoolikalt üle, mitte kiirustades, arvestades kuidas uus kood võib projekti ja teisi funktsioone muuta

Merge requesti kommentaarides võidakse tõstatada küsimusi, esitada ettepanekuid muudatusteks või anda üldist tagasisidet.

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