[Admiral16]

Hej Mirki od #programowanie potrzebuję porady. Przejęliśmy spory projekt mobilny w celu dalszego rozwoju i maintenance. Wstępny rzut oka architekta sugeruje, że kod ma sporo problemów i błędów. Czy znacie jakieś dobre, opensourcowe narzędzie, w którym ktoś mógłby zrobić review całego kodu i wskazać miejsca, które należy poprawić? W normalnym developmencie używamy mechanizmu merge-requestów, ale tutaj całość kodu już jest w repo, a nie wyobrażam sobie zgłaszania każdego niepoprawnego kawałka kodu jako ticketa do Jiry...

[LeniwiecPracoholik]

@Admiral16: jak działa nie ruszać (⌐ ͡■ ͜ʖ ͡■)

[zarev]

@Admiral16: nie ruszac, obserwowac ( ͡° ͜ʖ ͡°)

[Admiral16]

@LeniwiecPracoholik: Niestety, słabo działa, budżet na refactoring jest więc trzeba to zrobić - nie mam jednak koncepcji jak kwestię "sprzątania" podzielić pomiędzy członków zespołu...

[xaro]

Najpierw odpalcie oddupiacz. #pdk

[Koloses]

Napiszcie od nowa, mniej nerwów xD

[Admiral16]

@Koloses: Nie wchodzi w grę - musimy wprost wskazać, co i gdzie trzeba poprawić

[M.....k]

@Admiral16: Nie ma narzędzi które magiczne robią review kodu, gdyby były to nie potrzebowalibyście tego architekta :P Co najwyżej możecie poszukać narzędzi do statycznej analizy kodu w waszym języku co pewnie pozwoli wyłapać kilka błędów. W drugiej kolejności musicie po prostu znaleźć kogoś kto ma doświadczenie w architekturze i wam powie co macie robić.

[Admiral16]

@MDobak nie chodzi o magię tylko o sposób przekazania wyników - architekt leci przez kod i zaznacza miejsca do poprawy tak, żeby zespół mógł je zobaczyć - normalnie jak ktoś robi MR, leader wskazuje rzeczy do poprawy i akceptuję po ich poprawieniu, tutaj chodzi o to, żeby architekt nie musiał robić ticketa dla każdego znalezionego problemu tylko żeby je jakoś oznaczył w kodzie bez robienia bałaganu na repo. Ostatecznie myślałem o forku,

[inquis1t0r]

@Admiral16: Sonarem czy innym linterem przelecieć?

[Admiral16]

@inquis1t0r kurcze, Rrview ma kto zrobić - pytanie jak ma zgłosić problemy developerom

[M.....v]

@Admiral16: A może oddzielny pusty branch, Pull Request na tego brancha na Githubie/ Bitbuckecie/ GitLabie.

Dostaniesz na (chyba) każdej platformie wszystkie pliki i możliwość komentowania w nich i zaznaczania zmian.

A potem będzie możliwość dalszego commitowania, aż wszystko zostanie poprawione.

[Admiral16]

@Maav myślałem o tym, ale tutaj ktoś kto robi review będzie pracował na kodzie a nie na jego historii - czyta kod i zaznacza błędy. Patrzyłem na Gerrita, ale podobno toporny jest

[M.....v]

@Admiral16: Ale jak robisz Pull Requesta, to sprawdzasz najbardziej aktualne pliki. Tam nie ma żadnej historii.

Będzie to trochę niewygodne, jeśli chodzi o przechodzenie między plikami i szukanie zależności.

Ale chyba VS Code ma jakieś Extension do GitHuba https://code.visualstudio.com/blogs/2018/09/10/introducing-github-pullrequests .Wtedy miałbyś the best of both worlds. Nie korzystałem z tego osobiście, więc tylko podrzucam pomysł do sprawdzenia.