Zasady pracy

Git workflow

1. Pull Request (najlepiej do środy)
   • MAŁE zmiany (do 5 plików i 300 linijek, chyba, że generowane - wtedy napisać które pliki są generowane automatycznie)
   • W PR jako Assignees oznaczamy Bohdana lub Tomka, którzy wyznaczają osoby do review
   • Review powinno zająć do 20-30 minut
   • Reviewer musi odpalić kod
   • Od 1 do 2 reviewerów
     • Wybieramy na podstawie listy dostępności Idealnie:
     • Jedna osoba co jest krócej w projekcie
     • Jedna osoba co jest dłużej
   • Komentarze w PR mile widziane jeśli kod jest trudny, link z issue jeśli kopiujemy z stackoverflow
2. Reviewerzy mają 1 dzień na zrobienie review
   • Reviewerzy są odpowiedzialni za pingowanie twórcy, żeby wprowadził zmiany
   • Twórca ma 1 dzień na odniesienie się do komentarzy (zmiany może wprowadzić później)
3. Assignee merguje PR do developa; robiąc merge ze squashem

Jeśli PR jest specyficzny, można nagiąć zasady !

Przykazania

1. Nie przeprowadzamy refactoringu. Jeśli uważamy, że refactoring jest konieczny to dodajemy taska do bugów.
2. Jeśli zauważamy buga to dodajemy go na Tarze. Uprzednio upewniamy się, że ten błąd nie został już wcześniej wychwycony.
3. Przy mergowaniu używamy squasha.