Что не пропустишь на Merge Request

1. Отсутствие тестов – если код добавляет новую функциональность или меняет логику, тесты обязательны.
2. Грязный код – плохие названия переменных, дублирование, нарушение SOLID/DRY.
3. Неочевидные изменения – если в MR есть скрытые правки (например, изменение конфигов), их нужно явно описать.
4. Критичные ошибки – падения, утечки памяти, проблемы с производительностью.
5. Отсутствие ревью у зависимостей – если MR меняет package.json или другие зависимости, их тоже нужно проверить.

Пример плохого кода, который не пропущу:

// Плохо: магические числа, дублирование
function calc(a) {
  return a * 3.14 + 5;
}

Лучше:

const TAX_RATE = 3.14;
const FIXED_FEE = 5;

function calculateTotal(amount) {
  return amount * TAX_RATE + FIXED_FEE;
}