Sprawdzanie zmiany

Ten dokument zawiera instrukcje sprawdzania zmiany. Pamiętaj o tych kryteriach prawidłowej zmiany:

  • Czy ta zmiana jest zgodna z zamierzonym celem projektu?
  • Czy ta zmiana jest prawidłowa w ramach istniejącej architektury projektu?
  • Czy ta zmiana wprowadza błędy projektowe, które mogą powodować problemy w przyszłości?
  • Czy ta zmiana jest zgodna ze sprawdzonymi metodami obowiązującymi w tym projekcie?
  • Czy ta zmiana to dobry sposób na wykonanie opisanej funkcji?
  • Czy ta zmiana wprowadza jakieś zagrożenia dla bezpieczeństwa lub niestabilności albo zakłóca działanie istniejących aplikacji?

Więcej informacji o sprawdzaniu zmian znajdziesz w sekcji Sprawdzanie zmiany w dokumentacji Gerrit.

Jeśli masz rolę weryfikatora zmiany, otrzymasz e-maila z linkiem do zmiany. Aby przejrzeć i sprawdzić wprowadzone zmiany:

  1. Kliknij link do zmiany w e-mailu. Pojawił się ekran Gerrit's Change Info. Na tym ekranie znajdziesz opis zmiany, autora zmiany i wszystkich weryfikatorów oraz listę plików powiązanych z tą zmianą.

  2. Przeczytaj opis zmiany, aby dowiedzieć się, jaki jest jej cel.

  3. Kliknij plik, aby przejrzeć wprowadzone w nim zmiany. Plik zostanie otwarty na dwóch panelach i na ekranie porównania z istniejącym plikiem po lewej stronie, a plik z proponowanymi zmianami po prawej. Aby sprawdzić plik:

    1. Za pomocą myszy zaznacz (kliknij i przeciągnij) fragment kodu, do którego chcesz dodać komentarz lub pytanie. Pojawi się komunikat „Aby skomentować, naciśnij c”.

    2. Naciśnij C. Pojawi się pole do edytowania wersji roboczej komentarza.

    3. Wpisz komentarz lub pytanie i kliknij ZAPISZ.

  4. Kliknij unikalny numer zmiany (w lewym górnym rogu ekranu), aby wrócić do ekranu zmiany informacji.

  5. Kontynuuj sprawdzanie każdego pliku objętego zmianą.

  6. Gdy sprawdzisz wszystkie pliki objęte zmianą, wróć do ekranu Informacje o zmianie i kliknij ODPOWIEDZ. Pojawi się ekran przesyłania głosów na potrzeby wymagań.

  7. Wybierz jeden z następujących wyników dla zmiany:

    • +1 Wygląda mi dobrze, ale ktoś inny też musi się zgodzić na to.
    • 0 Brak wyniku

    • -1 Wolę, aby ta informacja nie była przesyłana w takiej postaci, w jakiej jest, ale można to zmienić.

    • -2 Nie należy przesyłać tej zmiany

  8. Kliknij ODPOWIEDZ, aby wysłać komentarz i ocenę do dewelopera zmiany.

Jeśli są jakieś problemy do naprawienia, deweloper je eliminuje i prosi Cię o przeczytanie poprawek. W przeciwnym razie sprawdzanie zmiany zostanie zakończone.