crnkjck
commented
4 years ago Pozeral som diffy a zacina to byt neprehladne. Zda sa, ze kvoli zmene sposobu commitovania sa pomenili aj konce riadkov z unixovych na windowsove alebo naopak, takze sa mi tazko zistuje, co sa zmenilo.
Kod inak vyzera dost dobre. Pridam zopar pripomienok, ale chcel by som to co najskor merge-nut do master, aby nam tu nenabublavali obrovske diffy.
Vidim, ze jazyk a factories som zorganizoval zle. Arita by mala byt v jazyku a parser by ju mal kontrolovat sam. Factories by uz mali byt naozaj iba volania konstruktorov bez testov. Potom by sa factories inicializovali raz ako konstantny objekt, ktory sa da vsade recyklovat, kym teraz sa musi stale vyrabat (stacilo by sice iba pri zmene jazyka, ale to nie je take podstatne).
Mergeol som staršie pull requesty do vetvy
mastera vetvuparser-implementationsom „presadil“ nad novýmaster. Malo by teraz byť bezpečné naklonovať si tento repozitár, prepnúť sa do vetvyparser-implementationa prekopírovať aktuálne verzie súborov. Následne už commitujte cez git, nie uploadovaním.Môžete použiť GitHub Desktop, ale väčšina programátorkých editorov a IDE má podporu pre git – sledovanie zmien, commitovanie, prepínanie vetiev, atď. Prečítajte si nejaký tutoriál/pozrite video k používaniu gitu s vaším IDE a začnite ho aktívne používať. Potom ani nepotrebujete GH Desktop – ten nakoniec vraj aj tak toho až tak veľa nevie; na vaše účely by podpora priamo v editore/IDE mala stačiť.