@matcheek: zmień jeszcze pasek na różowy, bo trochę niepoważnie to brzmi.
Review jest od pilnowania jakości kodu, w tym formatu. Jeśli uzgodniony format używa spacji, trzeba się tego trzymać. Jakość jest bardzo łatwo zepsuć - a pierwszy wyłom szybko pociąga za sobą następne...
@matcheek: format kodu ma ogromne znaczenie podczas pracy z gitem czy jakimkolwiek innym vcs. Wyobraź sobie ,że robisz reformat codu przy okazji jakiejś zmiany i nagle w commicie ląduje informacja o dziesiątkach/setkach linii które nie dotyczą faktycznej zmianie w kodzie. Takie commity są bezwartościowe, bo utrudniają code review i ustalenie co faktycznie się zmieniło w kodzie.
@koalapower: W Pythonie i Haskellu, OK, ale nie w .NET. Komu robi roznice czy masz cztery spacje, czy jeden tab w .NET? Pozatym programy do diffów mozna ustawic tak by ignorowaly whitespace.
Czekalem na ten moment od kilku miesiecy :)
#programowanie #codereview #korposwiat
Review jest od pilnowania jakości kodu, w tym formatu. Jeśli uzgodniony format używa spacji, trzeba się tego trzymać. Jakość jest bardzo łatwo zepsuć - a pierwszy wyłom szybko pociąga za sobą następne...
Komu robi roznice czy masz cztery spacje, czy jeden tab w .NET? Pozatym programy do diffów mozna ustawic tak by ignorowaly whitespace.