Komentarze (13)

@redve semantyka nie ma związku z ilością kodu jaki wrzucasz tylko jego działaniem. z grubsza to wygląda tak:


  • MAJOR version when you make incompatible API changes

  • MINOR version when you add functionality in a backward compatible manner

  • PATCH version when you make backward compatible bug fixes

chyba, ze chodzi ci o ilosc zmian jak na jeden commit. to wtedy faktycznie ktos grubo przesadzil. z drugiej strony jest to normalne na wczesnych etapach projektu

Ja za to robię commit co zmianę w klasie. Potem w PR jest zmodyfikowane 13 plików i 80 commitów. Jakieś takie przywyczajenia z Quick Save w Duke Nukem 3D ;)

@Trawienny próbowałem tak kiedyś i to jest przegięcie w drugą stronę. Potem przy wprowadzaniu poprawek robienie rebase i amend często trwa więcej czasu niż rzeczywista zmiana (bo zmieniałeś np jeden znak). W przypadku zmian w klasach często też będą występować konflikty, bo zmiana w klasie może nieść za sobą zmiany w innych plikach w użyciach tej klasy. Spotkałem się też ze stylem commit na każdy plik, wtedy konfliktów raczej nie ma, ale wprowadzenie jakiejś jednej ogólnej zmiany powoduje kilka commitów.

Wniosek: commituj jak chcesz a potem przy mergu rób squasha xd

To nie jest śmieszne nic a nic, jak jeszcze chodziłem do biura to biłem kijem programistów co wrzucali więcej niż 500 linii zmian

Zaloguj się aby komentować