[budyn]

Tak długo pracuję, ale dopiero teraz rozkminiłem (bo mi się przydarzyło), że PR-a trzeba przetestować (tak, odpalamy a nie tylko code review) ze zmergowanym target branchem.
Pierwszy przypadek miałem taki, że osoba A pracowała na 2 plikach, osoba B na 2 innych.
Każdy z osobna branch działał ok. Ale razem się rozjebują w runtimie.
Czyli, dopiero jak oba PR zmergowałem to się #!$%@?ł target branch.

Build (kompilacja) przechodził, ale javascripty się #!$%@?ły.

Może dla niektórych to oczywiste, ale ja się dzisiaj czegoś nowego nauczyłem.

#git #github #programowanie

[vytah]

ale javascripty się #!$%@?ły.

@budyn: To akurat jest naturalny stan dla większości javascriptów, nie wiem skąd zdziwko

[villager]

@budyn: mozna chyba ustawic na githubie ze testy (których nie masz ?) zawsze muszą się odpalić na zrebaseowanej wersji pull requesta zanim mozna zmergowac. chyba ja tak mam teraz w korpo

[S.....i]

@budyn: coś musi być bardzo nie tak jak tak się stało ( ͡° ͜ʖ ͡°) zalecenie lekarza refactor

[budyn]

@villager: U nas nawet dwa buildy na PR idą, zrebasowany i osobno branch. Jakbyśmy mieli testy jsa to by tematu nie było.
Śmiesznie się robi jak czasami wisi kilka/naście PRów to po zmergowaniu jednego wszystkie inne się muszą odświeżyć (zrebasować)

[Hauleth]

@budyn: https://bors.tech