jam231 / sia

Stock market server (part of stock market simulation system).
1 stars 0 forks source link

Refactoring. #20

Closed marimarek closed 11 years ago

marimarek commented 11 years ago

Jesli cos mi sie nie pdoba/zrobilbym inaczej zmieniac to, pytac sie czy zostawiac? Swoja droga nie wiem, czy komentowanie starego kodu ma sens, bo korzystajac z gita mamy i tak cala historie zmian

jam231 commented 11 years ago

Jeżeli chcesz coś zmieniać to zrób nowego brancha, w którym będziesz sobie grzebał, a gdy stwierdzisz, że skończyłeś (czyli również przetestowałeś jakoś sensownie) to dasz znać i reszta zrobi review, i wtedy bedzie merge. Na obecną chwilę master jest up-to-date - experimental ma status wlasnie takiego testowego brancha. Myślę, że gdy dwie lub więcej osób będzie zmieniało codebase serwera na jednym branchu to powstanie chaos i będziemy sobie wchodzili w drogę, co będzie wywoływało frustrację (jak przy tym nieszczęsnym pliku config).

marimarek commented 11 years ago

To ja bym proponowal tez usunac tego expiremntal branch-a(albo zostawic go dla Ciebie) i niech kazdy ma swoj wlasny developerski branch, ktory po skonczonych zmianach i zaakceptowaniu bedzie mergowany i usuwany. Jakby byly jeszcze jakies wieksze feutur'y do zrobienia to tez proponuje je robic na innym branchu, dzieki temu mozna tam wrzucac jeszcze niedokonca dzialajace feutur'y, a ktos inny jednoczesnie na innym branch'u moze robic zupelnie cos innego, a branch master bedzie zawsze branchem z najnowasza dzialajaca wersja. Jesli po rozbranchowaniu osoby beda pracowac nad innymi kawalkami kodu to merge odbedzie sie bezbolesnie bez recznego scalania.

kaiks commented 11 years ago

w teorii to brzmi fajnie, ale zdaje mi się że Was poniosło i w powstającej w ten sposób biurokracji praca po prostu nie będzie szła sprawnie (a zamiast spędzać czas na pisaniu będziemy go spędzać na bawieniu sie z githubem i czekaniu na wzajemne akceptacje)

raczej bym proponował żebyśmy się zajmowali różnymi kawałkami kodu, ale ja i tak na obecnym etapie robie raczej sql+agentów niż serwer, więc jak tam chcecie

jam231 commented 11 years ago

Ok niech tak, będzie. Experimental pozostanie moim deweloperskim branchem do zabawy, etc.