Upload
zsolt-huba
View
279
Download
2
Embed Size (px)
Citation preview
Code Review Gerittel
Huba [email protected]
Gerrit
● GWT webalkalmazás● Gatekeeper, git szerver is egyben● Integrálódik CI szerverrel● Eclipse támogatás
● Git plugin● Mylyn
Miért volt szükségünk rá?
● Subversion műveletek távoli szerverre lassúak● Revertet leszámítva minden más művelettel a
szerverhez fordul● Subversion branchelés nehézkes● Igény az elosztott verziókezelők
funkcionalitására● Igény code review eszközre● Kiadott release csak ellenőrzött kódot
tartalmazzon
Projekt git configja
Change ID
● Review egyedi azonosítója● Commit hook generálja● Git log utolsó sorában jelenik meg
Fejlesztés menete
● Lokális feature branch (1 feature / branch)● Nincs merge, csak rebase a masterre →
tisztább history● git push
● Egy commit/beküldött change (1 Change ID)● Beküldés előtt szükség szerint:
– git rebase i > squash● Érdemes elkerülni az egymásra épülő change-
eket● Újranyitott review esetén git commit amend
Gépi review (CI)
● Hudson / Jenkins végzi● git push után automatikusan indul
● Gerrit Trigger● Értékei:
● +1 Verified● 0 No score● -1 Fails
Nyitott review-k
Kézi review
● Beküldő reviewert rendel a beküldött change-hez
● Csak akkor van értelme átnézni a change-et, ha átment a gépi review-n
● Code Review ● webes felület vagy
Eclipse● Reviever megjegyzéseket
fűz a kódhoz
Diff nézet
Kézi review kimenete● Change pontozása● (Reviewer visszadobja
a change-et)● Újabb beküldés
tetszőleges számú alkalommal
● Reviewer elfogadhatja change-et
● Sikeres review után merge a master ágba● Többi fejlesztő számára
ilyenkor lesz elérhető
Kérdések / Válaszok