Przedstawiamy prośby Pull dla zespołu 2-osobowego - scalić moje własne prośby?

11

Przedstawiam git młodszemu członkowi zespołu (spółdzielni).

Są teraz wygodne z podstawami dodawania, popełniania, pchania i ciągnięcia.

Teraz chcę je przedstawić, aby pobierać żądania i gałęzie.

Jeśli zaczną robić żądania ściągania w oddziałach, czy powinienem zrobić to samo dla mojej bieżącej pracy?
Będę tym, który połączy ich żądania ściągania. Nie byłem pewien, czy praca z oddziałami byłaby dla mnie sensowna (ogólnie dobra praktyka, którą znam, ale jestem ciekawa tej konkretnej sytuacji 2 deweloperów z jednym młodszym ), a jeśli tak, to znaczy, że po prostu połączę własne gałęzie w master. Czy mimo to wykonałbym żądanie ściągnięcia mojej pracy / oddziałów? Zasadniczo do tych zmian używamy podstawowego przepływu pracy gałęzi funkcji github:
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

Czy istnieje cel używania żądań ściągania na moim własnym repozytorium, jeśli jestem jedynym programistą? jest przydatne, ale nie tak szczegółowe.

Jaki przepływ pracy z 2 osobami przy projekcie wydaje się również bardziej ogólny

i

Czy powinienem otwierać żądania ściągania z oddziału na oficjalnym repozytorium lub moim forku? wydaje się więcej o widelcach.

Michael Durrant
źródło

Odpowiedzi:

19

Nie. Nie powinieneś scalać własnych żądań ściągania. To, co jest dobre dla gęsi, jest dobre dla gąsiora. Łączenie własnych próśb ściągania stanowi zły precedens dla naszego młodszego programisty. Oznacza to również, że nikt inny nie patrzy na twój kod. Bez względu na to, jak zaawansowani jesteśmy, wszyscy od czasu do czasu popełniamy błędy i piszemy zły kod. Naucz swojego młodszego, jak działają recenzje kodu z drugiej strony, zmuszając go do przeglądu i scalenia twojej pracy.

Może nie ma tego samego oka, co ty, ale dzięki temu przyzwyczai się do procesu od końca recenzenta i może cię zaskoczyć i złapać coś głupiego, co zrobiłeś. Minimalnie da ci wskazanie fragmentów kodu, które są dla ciebie oczywiste, a które nie są dla niego oczywiste. Ma to podwójną korzyść.

  1. Oboje uczycie się, gdzie twój uczeń musi skoncentrować się na nauce.
  2. Uczysz się, gdzie jesteś mądrzejszy niż powinieneś.
Gumowa kaczuszka
źródło
6
Inną ogromną zaletą recenzji kodu jest to, że co najmniej dwie osoby widziały, wiedziały i miały możliwość zadawania pytań na temat każdej zmiany kodu przed jego wprowadzeniem. Nawet jeśli młodszy programista nie wie, czego szukać, jest z pewnością nauczy się czegoś z tego wszystkiego.
Ixrec,