Utrzymuję stosunkowo popularne repozytorium github.
Kiedy żądanie ściągnięcia jest dobre do scalenia, zwykle pytam autora o zmianę bazy na pojedyncze zatwierdzenie przed scaleniem (szczególnie gdy było wiele małych edycji).
Czy to dobra praktyka git? Czy jest to akceptowana / standardowa etykieta GitHub?
Więc niektóre korzyści:
- Otrzymuję ładną, czystą historię zatwierdzeń w dziennikach zatwierdzeń
- Nie muszę samodzielnie zmieniać zatwierdzenia
- Deleguje część pracy
Niektóre możliwe wady:
- Nie jestem pewien, czy to dobra etykieta
- Nie jestem pewien, czy to dobra praktyka git
- Zwykle prosiłem już o kilka innych zmian - to jeszcze jedna i nie chcę zniechęcać współpracowników.
git push -f
?Odpowiedzi:
Jeśli chodzi o Git, to jest coś w rodzaju świętej wojny, niezależnie od tego, czy powinieneś po prostu scalić gałęzie, czy też zatwierdzić zmiany w najnowszej wersji gałęzi, do której się przyłączasz. Istnieje wiele rozmów na temat tego, co jest lepsze, jeśli przeprowadzasz szybkie wyszukiwanie w Programmers.SE .
Co się tyczy etykiety, poradzimy sobie z tym z praktycznego punktu widzenia. Kiedy mamy do czynienia z nowym kodem pochodzącym od kogoś innego, zawsze najlepiej jest zmusić go do scalenia najnowszych zmian z oddziału lub odświeżenia go przed scaleniem, aby zapewnić czyste scalenie. Pamiętaj, że napisali kod, więc są zwykle najbardziej wykwalifikowani do rozwiązywania wszelkich konfliktów scalania / rebase. Osobiście nie widzę z tym problemu i cały czas widzę tę prośbę od innych osób. Dla mnie, jeśli nie ma żadnych konfliktów, często zrobię to sam, ponieważ jest to dwusekundowa aktualizacja, którą git może zastosować samodzielnie. Ale jeśli wystąpią konflikty, zawsze poproszę oryginalnego autora kodu, aby sam sobie z nim poradził.
Ponadto, w przypadku GitHub (przynajmniej), będą wyświetlać link do twojego
CONTRIBUTING
pliku ponad wszelkimi próbami PR, dzięki czemu jest to dobre miejsce do nakreślenia twoich oczekiwań, a wiele projektów zawiera to, że będą łączyć tylko aktualne gałęzie.źródło