Z tego dokumentu dowiesz się, jak sprawdzić zmianę. Pamiętaj o tych kryteriach prawidłowej zmiany:
- Czy ta zmiana jest zgodna z określonym celem tego projektu?
- Czy ta zmiana jest zgodna z dotychczasową architekturą projektu?
- Czy ta zmiana wprowadza wady projektu, które mogą w przyszłości powodować problemy?
- Czy ta zmiana jest zgodna ze sprawdzonymi metodami, które zostały ustalone w tym projekcie?
- Czy ta zmiana jest dobrym sposobem na wykonanie opisanej funkcji?
- Czy ta zmiana wprowadza jakiekolwiek zagrożenia dla bezpieczeństwa lub stabilności albo powoduje nieprawidłowe działanie istniejących aplikacji?
Więcej informacji o sprawdzaniu zmian znajdziesz w sekcji Sprawdzanie zmian w dokumentacji Gerrit.
Jeśli zostaniesz wyznaczony(-a) do sprawdzenia zmiany, otrzymasz e-maila z linkiem do niej. Aby sprawdzić i zatwierdzić zmianę, wykonaj te czynności:
Kliknij link do zmiany w e-mailu. Wyświetli się ekran Gerrit's Change Info (Informacje o zmianach w Gerrit). Na tym ekranie znajdziesz opis zmiany, informacje o osobie, która ją wprowadziła, oraz o wszystkich recenzentach, a także 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 w 2 panelach. Po lewej stronie będzie widoczny istniejący plik, 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 pytanie. Pojawi się komunikat „Naciśnij C, aby dodać komentarz”.
Naciśnij C. Pojawi się pole wersji roboczej komentarza.
Wpisz komentarz lub pytanie i kliknij ZAPISZ.
Kliknij unikalny numer zmiany (w lewym górnym rogu ekranu), aby wrócić do ekranu Informacje o zmianie.
Kontynuuj sprawdzanie każdego pliku w zmianie.
Po sprawdzeniu wszystkich plików 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 Wolę, żeby nie przesyłać tego w obecnej formie, 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 usuwa i prosi Cię o sprawdzenie poprawek. W przeciwnym razie sprawdzenie zmiany jest zakończone.