Jak na code reviews na GitHubu rubrika: Programování: Jiné

2 borekb
položil/-a 15.10.2015
 
upravil/-a 15.10.2015

Přecházíme z Bitbucketu na GitHub a hledáme způsob, jak tam organizovat code reviews u pull requestů.

Na Bitbucketu to funguje zhruba takhle:

  1. Otevřu pull request
  2. Přidám své kolegy jako reviewery, ti dostanou notifikaci
  3. Kolega PR zkoukne a:
    3a. Dá fajfku že je to za za něj OK
    3b. Napíše nějaké komentáře, případně z komentářů rovnou vytvoří "tasky" (nejsou to plnohodnotné issues, jen seznam checkboxů přímo na PR, aby se na nic nezapomnělo)
  4. V přehledu (tabulce) PRs vidím stav každého PR - kolik reviewerů na něm je, kolik z nich už dalo fajfku a jestli tam jsou nějaké tasky k dořešení

Je to dost přirozené a chtěli bychom toto flow zachovat i na GitHubu, ale pokud se nepletu, umí jen 1) a 3b). Jak se tam řeší, aby se člověk zaprvé dozvěděl, že se na nějakém PR stal reviewerem, a pak samotný akt schválení (fajfka)? Napadá mě nějak využít labely, ale připadá mi to dost neohrabané. Konkrétní tipy vítány, díky!

Komentáře

  • jednabedna : Docela by me zajimalo co vas k tomu prechodu vede. Myslis, ze by ses mohl podelit? — 15.10.2015
  • devnull : Nevím, jak autora, ale mě na BitBucketu vadí: 1. výpadky (asi před měsícem celodenní, od té doby pár menších), 2. skoro nulový vývoj (#4285 a #8672 sleduju už rok a žádná snaha to řešit), 3. chybějící integrace u některých služeb (třeba z CI má BB jen pár, GH všechny). Ale stejně je pro nás cenová politika GitHubu tak nevýhodná, že u BitBucketu zůstáváme. — 15.10.2015
  • borekb : Podobně jako devnull. Jednička nás až tak netrápí, ale dva a tři potvrzuju, plus komunitní vývoj v podstatě musí probíhat na GitHubu, protože na BB lidi nejsou. — 15.10.2015
  • martenson : 2) zminim (@borekb) kazdeho od koho chci feedback 4) Mame primitivniho bota na jenkins, ktery pocita "+1" (favorable reviews) a pokud jich je dost napise do PR zpravu, ze je povolen merge (nemuzu dat odkaz kvuli repotacu takze pro priklad mrknete na repo galaxyproject/galaxy a PR 885) — 15.10.2015
odkaz
3 tomas.fejfar
odpověděl/-a 15.10.2015
 
upravil/-a 15.10.2015

1) funguje stejně
2) V detailu PR jsou "asignees"
3) a) :+1: v komentáři b) inline comments + "- [ ] text tasku"
4) AFAIK nejde

Jinak z labelů používáme WIP (tj. nehotový PR, ke kterému se ale dá komentovat spíš koncepčně.

Co se týče přímo toho procesu, tak za merge PR je zodpovědnžý autor, který když mu jich moc visí, tak si musí reviewery propingovat buď tam nebo na Slacku/Hangouts/naživo. Snažíme se navíc dělat PR tak malé, aby stačil 1 reviewer a aby mu review zabral < hodinu. Občas se to sice nepovede, ale pak se za ten mega PR všichni svorně stydí a snaží se ho dotlačit k reviewnutí a mergnutí.

Komentáře

  • borekb : Díky. Ty tasky (checkboxy) musím lovit po těle pull requestu nebo je GitHub vytáhne někam nahoru? — 15.10.2015
  • Grelek : Bohužel mám pocit, že jsou řazené jako všechny ostatní komentáře, takže to není moc přehledné. — 15.10.2015
  • hrachcz : Asi neni problem, aby revieweri editovali uvodni comment a pridavali tasky do neho. — 15.10.2015
  • tomas.fejfar : Editujeme úvodní comment — 15.10.2015

Pro zobrazení všech 2 odpovědí se prosím přihlaste:

Rychlé přihlášení přes sociální sítě:

Nebo se přihlaste jménem a heslem:

Zadejte prosím svou e-mailovou adresu.
Zadejte své heslo.

Aktuální otázky v rubrice Programování: Jiné

Otázky uživatele borekb