Czy istnieje dobry sposób na wprowadzenie recenzji?
Prawdopodobnie istnieje kilka dobrych sposobów, w zależności od zespołu i korzyści, które można uzyskać z recenzji, ale każde podejście będzie miało kilka wspólnych cech:
wyjaśnij, czego oczekujesz: jest to nowy proces dla Twojego zespołu lub przynajmniej zmiana w istniejącym procesie, więc uczciwe jest jedynie poinformowanie zespołu, dlaczego wprowadzasz zmianę, jak możesz oczekiwać, że zespół odniesie korzyść, i skąd będziesz wiedzieć, czy to działa.
zdefiniuj proces: Przeprowadź ludzi przez proces, który chcesz, aby sprawdzili kod, omawiali zmiany itp., aby wszyscy w zespole wiedzieli, jak postępować.
Zdefiniuj kryteria: Określ rodzaje zmian, które ludzie powinni, a nie powinni nazywać, że wymagają poprawy. Na przykład warto zauważyć błędy i znaczące ulepszenia wydajności; standardy kodowania, czytelność i łatwość konserwacji powinny być odnotowane, ale nie rozwiązywane; kwestie osobistego gustu lub stylu należy pozostawić w spokoju.
omów zachowanie: Zwróć uwagę, że celem jest poprawa kodu i wspólne zrozumienie, które pomoże zespołowi pisać lepszy kod na forum, nie zawstydzać nikogo, ustalać wyniki itp. Krytyka powinna być obiektywna i konstruktywna, nigdy osobista. Ustanowienie niektórych podstawowych zasad może pomóc złagodzić wątpliwości związane z przeglądem kodu.
postaw się na pierwszym miejscu: Niezależnie od tego, czy planujesz mieć indywidualne recenzje czy recenzje grupowe, prawdopodobnie dobrym pomysłem jest przejrzenie kilku pierwszych jako grupy. Pierwsza recenzja powinna zawierać Twój własny kod, aby inni członkowie zespołu mogli zobaczyć, że proces nie jest taki zły i że jesteś gotów sam go wykonać.
Rozpocznij od spotkania inauguracyjnego, aby wyjaśnić wszystkie powyższe kwestie i odpowiedzieć na obawy członków zespołu. Następnie prześlij wiadomość e-mail z dokumentacją procesu.
Wyczuwam dużą niechęć zespołu, ponieważ jest to jeszcze jedna rzecz do zrobienia, a rozmowy mogą być bolesne.
Są to dwie odrębne obawy. Jeśli uważasz, że recenzje będą pomocne, musisz poświęcić czas na ich wykonanie. Upewnij się, że członkowie zespołu rozumieją, że sprawdzanie działa jak każde inne zadanie, a nie coś dodatkowego, co muszą zrobić, kontynuując wykonywanie innych zadań w tym samym tempie.
Spotkania grupowe powinny być prowadzone przez moderatora, który prowadzi dyskusję, ogranicza długość spotkania i utrzymuje konstruktywność. To powinno pomóc w uniknięciu bolesnych rozmów. Do czasu, gdy będziesz gotowy do rozpoczęcia indywidualnych recenzji, zespół powinien przyjąć zachowania, które pomogą im zachować konstruktywność.
Od czasu do czasu należy również przejrzeć proces recenzji. Co jakiś czas zbieraj zespół, aby omówić proces: jak działa, jak można go poprawić, jakie praktyki należy porzucić itp. Daj zespołowi prawo własności do procesu i swobodę próbowania nowych rzeczy.