Hejto.pl
Dodaj post

Wpisz coś do wyszukania (minimum 2 znaki)

Wpis użytkownika redve w Hydepark

Fanatyk

w Hydepark

10


uwielbiam używać gita

Komentarze (13)

Zawodowiec0

kto reviewował takiego kloca 2k linii zmian ten w cyrku się nie śmieje.

Fanatyk3

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

Zawodowiec0

Komentarz usunięty

Tytan1

Yeah, to ja daję takie o nazwie "stuff I've been working on for past 3 months"

Gruba ryba3

Daj reviewerowi PR z 10 zmianami to znajdzie 4 błędy. Daj mu taki z 1000+ zmianami to nie znajdzie żadnego. Róbcie małe PR.

Gruba ryba2

Ja j⁎⁎ie, jak bym mógł to od razu reject na czymś takim.

Fanatyk2

@redve - nie takie rzeczy człowiek widział czy nawet sam odpierdalał :face_with_hand_over_mouth:

Fenomen3

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 ;)

Fanatyk0

@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

Gruba ryba0

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

Gruba ryba6

@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

Gruba ryba1

@redve tak samo robie, czasami daje "major changes" jak jest dużo xDD albo "general improvements"