mobilecodereviewer / MobileCodeReviewer

0 stars 0 forks source link

[S] Możliwość porównania zmian pomiędzy wybranymi patchsetami #37

Open fracz opened 9 years ago

fracz commented 9 years ago

Przykład: http://apps.iisg.agh.edu.pl:8081/#/c/25/3/app/src/main/java/pl/edu/agh/mobilecodereviewer/configuration/InjectionModule.java

W tej zmianie są trzy patchsety - w pierwszym usunąłem jeden komentarz, w drugim drugi a w trzecim dopisałem jedną linię kodu w innym pliku. Gerrit umożliwia teraz porównanie na diffie każdą wersję z każdą poprzez przyciski na górze.

To jest bardzo ważne gdy review "wraca" do reviewera - nie musi on sprawdzać wszystkiego od początku ale patrzy tylko na zmiany w kodzie wykonane od ostatniego review.

d00d171 commented 9 years ago

Drobna prośba o przejrzenie w pierwszej kolejności zgłoszeń z labelkami verification i question to reporter żeby zgłoszenia potencjalnie do zamknięcia nie zaśmiecały boarda.

fracz commented 9 years ago

Przeglądam od góry :-)