Z tego dokumentu dowiesz się, jak sprawdzić zmianę. Pamiętaj o tych kryteriach ważnej zmiany:
- Czy ta zmiana jest zgodna z określonym celem tego projektu?
- Czy ta zmiana jest prawidłowa w ramach obecnej architektury projektu?
- Czy ta zmiana wprowadza wady projektowe, które mogą powodować problemy w przyszłości?
- Czy ta zmiana jest zgodna ze sprawdzonymi metodami ustalonymi dla tego projektu?
- Czy ta zmiana jest dobrym sposobem na wykonanie opisanej funkcji?
- Czy ta zmiana wprowadza jakiekolwiek zagrożenia związane z bezpieczeństwem lub niestabilnością albo powoduje problemy z istniejącymi aplikacjami?
Więcej informacji o sprawdzaniu zmian znajdziesz w sekcji Sprawdzanie zmian w dokumentacji Gerrit.
Jeśli przypiszesz się do roli recenzenta zmiany, otrzymasz e-maila z linkiem do tej zmiany. Aby sprawdzić zmianę, wykonaj te czynności:
Kliknij link do zmiany w e-mailu. Wyświetli się ekran Informacje o zmianie w Gerrit. Zawiera on opis zmiany, informacje o jej autorze i wszystkich recenzentach oraz listę plików powiązanych ze zmianą.
Przeczytaj opis zmiany, aby zrozumieć jej cel.
Kliknij plik, aby sprawdzić zmiany w nim. Plik otworzy się na ekranie porównania z 2 panelami – po lewej stronie będzie plik w obecnej wersji, a po prawej plik z proponowanymi zmianami. Aby sprawdzić plik:
Za pomocą myszy zaznacz (kliknij i przeciągnij) fragment kodu, do którego chcesz dodać komentarz lub zadać pytanie. Pojawi się komunikat „Naciśnij c, aby dodać komentarz”.
Naciśnij c. Pojawi się okno roboczego komentarza.
Wpisz komentarz lub pytanie i kliknij ZAPISZ.
Aby wrócić do ekranu Informacje o zmianie, kliknij unikalny numer zmiany (w lewym górnym rogu ekranu).
Sprawdź każdy plik w zmianie.
Gdy skończysz sprawdzać wszystkie pliki w zmianie, wróć do ekranu Informacje o zmianie i kliknij ODPOWIEDZ. Pojawi się ekran Prześlij głosy dotyczące wymagań.
Wybierz jedną z tych ocen zmiany:
- +1 Wygląda dobrze, ale ktoś inny też musi zatwierdzić.
0 Brak oceny
-1 Wolałbym, żeby nie została przesłana w tej postaci, ponieważ są rzeczy do poprawienia.
-2 Ta zmiana nie powinna zostać przesłana.
Kliknij ODPOWIEDZ , aby wysłać komentarze i ocenę do dewelopera zmiany.
Jeśli są problemy do rozwiązania, deweloper je rozwiązuje i prosi o sprawdzenie poprawek. W przeciwnym razie możesz zakończyć sprawdzanie zmiany.