Narůstající pullrequesty na Githubu
- patriksima
- Člen | 58
Koukám na github Nette a vidím tam i rok staré pullrequesty. Přitom Hrach někdy reaguje i do pár minut (dík). Jde s tím něco dělat? Resp. můžeme my, obyčejní smrtelníci, používající Nette, nějak pomoct?
- Filip Procházka
- Moderator | 4668
Je potřeba, aby někdo s právy je prošel a buď zavrhl, nebo vyzval k rebasu na aktuální master a následně začlenil.
- Jan Tvrdík
- Nette guru | 2595
@enumag
Kdo všechno ale má práva (…)?
Vím jen o třech lidech, kteří mají push práva – David, Jakub Vrána a Hrach.
- enumag
- Člen | 2118
@Jan Tvrdík: Tak to jsme na tom stejně, o těch třech vím taky. :-) Navíc Vrana už neni aktivní a Hrach ta práva stejně nepoužívá (respektive jen na tagování issues a zavírání těch které implementoval v nějakém pull requestu). Takže jediný, kdo s tím může něco dělat je opravdu David.
- patriksima
- Člen | 58
paranoiq napsal(a):
K čemu to?
Což mi připomíná. Nemůže někdo znalý někam vyvěsit git(hub)/nette workflow pro opraváře chyb? Tzn. jak forknout, naklonovat repo, branch, rebase/merge, push, pull request…
Editoval patriksima (23. 1. 2013 11:04)
- hrach
- Člen | 1838
- ad postup pri PR: to uz sem ti popisoval. pripadne problemy se doresi v komentacich u konkretniho PR.
- ad prava: mel sem pocit, ze ma taky Filip Prochazka
- ad prava ja: dostal sem je (co vim) hlavne na ty issue, to, ze muzu pushovat je vedlejsi efekt. S Davidem jsem se o tom nikdy moc nebavil, nechcu do toho fusovat, jeho oko ma opravdu dobrou britvu, ktera Nette delam Nette :)
- Filip Procházka
- Moderator | 4668
Já práva rozhodně nemám. Ale kdo má jaké práva je úplně jedno, protože David musí vybrat co do Nette chce a co ne. I kdybych mohl pushovat, tak bych si rozhodně nedovolil někomu mergovat pullrequesty ;)
- patriksima
- Člen | 58
hrach napsal(a):
- ad postup pri PR: to uz sem ti popisoval. pripadne problemy se doresi v komentacich u konkretniho PR.
jasně, napsal. díky. ale ono by to chtělo asi i veřejně hned vedle
commit policy, aby to věděli všici, co se chtěj účastnit.
takhle to není moc developer friendly. i když otázka jestli se chce, aby do
toho fušoval kdekdo :)