Рев'ю Коду

Практика навчання команди проєкту під час якої досвідченіші розробники проглядають код своїх колег, допомагаючи зрозуміти точки його оптимізації і спрощення.

Правила проведення рев’ю коду:

• Переглядайте логіку реалізації функцій, шукайте речі, котрі не зможуть бути виявленими тестувальниками чи автоматизованими тестами, наприклад, скидування глобального кешу, котрий сповільнює роботу, але який складний для виявлення іншими способами.

• Віддайте перевірку стиля лінтерам (ESLint)

• Пріоритизуйте – якщо у Вас багато претензій до коду, оберіть декілька головних, які дозволяють вирішувати найважливіші завдання чи зменшити технічний борг.

• Архітектурні проблеми простіше/краще/менш праце затратно, обговорюючи способи реалізації до реалізації.

Негативним моментом неправильного рев’ю коду можуть стати конфлікти між розробниками, тому необхідно використовувати практику в дружній (рекомендаційній) формі.

Рекомендується не використовувати наказову форму (Роби ось так…) в знайдених дефектах (Краще…, Краще було б використати...).

Для мінімізації пінг понгу в обговорення коментаря використовують наступний принцип: Якщо розробник згоден з коментарем він вирішує проблему, якщо не згоден – підходить спілкуватися з рев’ювером особисто про виявлений дефект.

Додаткові джерела https://blog.softwareplanetgroup.co.uk/2018/11/01/making-the-most-of-code-review/ https://github.com/joho/awesome-code-review https://google.github.io/eng-practices/review/reviewer/

Організація проведення CodeReview https://www.youtube.com/watch?v=9meDxkow9V4