Как проходило Code Review в команде

В команде Code Review обычно проходит через Pull Request (PR) в Git (GitHub/GitLab/Bitbucket). Процесс:

1. Создание PR – разработчик делает ветку, пушит изменения, открывает PR с описанием изменений.
2. Ревью – коллеги проверяют код на:
   • Соответствие стандартам (ESLint, Prettier).
   • Оптимальность алгоритмов.
   • Читаемость и поддержку кода.
   • Тесты (если есть).
3. Комментарии и правки – ревьюверы оставляют замечания, автор вносит правки.
4. Апрув и мерж – после исправлений PR апрувится и мержится в основную ветку.

Пример хорошего PR:

// Было: сложный и непонятный код  
function calc(a, b) { return a + b; }  

// Стало: с JSDoc и проверками  
/**  
 * Складывает два числа.  
 * @param {number} a  
 * @param {number} b  
 * @returns {number}  
 */  
function sum(a, b) {  
  if (typeof a !== 'number' || typeof b !== 'number') {  
    throw new Error('Arguments must be numbers');  
  }  
  return a + b;  
}  

Важно: ревью должно быть конструктивным, без личных оценок.