niedorzecznybubr

Czepianie się gramatyki w commit message'ach na code review

  • rozum i godność człowieka 40.3% (58)
  • rak 52.1% (75)
  • nie mam zdania 7.6% (11)

144 głosy

  • Najlepsze
  • Wszystkie komentarze
Zimm3man
Zimm3man
Zimm3man
  • adult
10
Treść przeznaczona dla osób powyżej 18 roku życia...
dawid-mocki
dawid-mocki
@Zimm3man: przecież idzie squash merge z nazwą commita równą nazwie PRa, a message poszczegolnych commitow zostają zapomniane xd
niedorzecznybubr
Code Spell Checker, rozwiązuje ten problem,


@Kamilos01: niby jak? Mowa o commit message'ach, które pewnie 99% osób pisze w vimie
niedorzecznybubr
@sokytsinolop: wrzucasz na gerrita gotowy commit szczegółowo opisany. Jakieś wip to se możesz robić u siebie lokalnie a nie wrzucać gdziekolwiek
szefczyk
szefczyk
@niedorzecznybubr: *przepalanie czasu
niedorzecznybubr
@szefczyk: trzeba zwalczać niechlujstwo
Kamilos01
@niedorzecznybubr: Wydaje mi się, że jest to rzadki przypadek -> Code Spell Checker, rozwiązuje ten problem, gdzie nie wymaga praktycznie żadnego wysiłku
szefczyk
szefczyk
@niedorzecznybubr: half-broken jinglisz to jest przecież esencja pracy z pajetami, szkoda się przejmować - z fartem byczqu 🤙
s.....5
s.....5
to że tam napisze childs czy children to raczej nikogo to nie obchodzi


@Zimm3man: a jeśli w kodzie ma listę obiektów typu Child i nawie ją children zamiast childs? Co wtedy?
Zimm3man
Zimm3man
@sn3p5: To akurat bym może i wytknął na cr jakbym zauważył. Nazwy zamiennych czy metod to chyba jedyne do czego bym się doczepił. Ale o jakiś super poprawny opis commita bym się nie p-----------ł.
niedorzecznybubr
@dawid-mocki: co to za patologiczny system?
sokytsinolop
@niedorzecznybubr: squash merge. Gotowy feature trafia na docelowego brancha jako jeden gotowy commit a nie r-------e 150 commitow typu "wip, "in progress", "its working"
Kamilos01
@niedorzecznybubr: "Pewnie 99%" -> skąd takie przypuszczenia? W żadnej z firm w której pracowałem, lub miałem okazję pracować jako kontraktor (ew. nawet patrząc po znajomych z branży) nikt nie robił tego w ten sposób -> większość korzystała z rozwiązań wbudowany w IDE. Więc w takim przypadku to rozszerzenie rozwiązuje ten problem, ale oczywiście zgadzam się z Twoją myślą, jeżeli faktycznie tak jest, to moje rozwiązanie nie pomoże :)
niedorzecznybubr
@Kamilos01: moze jeszcze na Windowsie pracujecie xd
dawid-mocki
dawid-mocki
@niedorzecznybubr: nie wiem, funkcja githuba, azure repos, pytaj producentów xd
niedorzecznybubr
@dawid-mocki: ja wiem że tak działa to na githubie, ale to nie jest poważny podejście moim zdaniem. A każda zmiana powinna być szczegółowo opisana (po co to, jeżeli coś jest zrobione dziwne to czemu tak, itd)