Процедуры контроля и автоматизация
Автор PR обязан убедиться, что
- PR содержит только обновление / миграцию — никаких посторонних изменений
- CI пройден полностью — все шаги зелёные
- Описание PR содержит:
- Обоснование — зачем обновляемся (безопасность, производительность, новая функциональность)
- План отката — как вернуть изменения
- Описание breaking changes и миграционных шагов
- Скриншоты / видеозаписи до и после — при наличии визуальных изменений
- План отката предварительно протестирован
Ревьюер обязан убедиться, что
Процесс миграции:
- PR не содержит несвязанных изменений
- Изменения изолированы
- План отката понятен и реалистичен
- Legacy API помечены
@deprecatedс указанием альтернативы
Безопасность изменений:
- Отсутствие регрессий на основании результатов CI
- Скриншоты / видеозаписи до и после соответствуют ожиданиям (при наличии визуальных изменений)
- Покрытие тестами сохранено (если применимо)
Автоматизация
Renovate Bot / Dependabot
MUST
- Автоматические PR для minor-обновлений.
- Changelog в описании PR.
- CI-проверка на каждое обновление.
Процесс
| Тип обновления | Кто мержит |
|---|---|
| Minor | Любой член команды после прохождения CI |
| Major | Требуется апрув Tech Lead проекта |
ESLint — Запрет импорта Legacy API
// .eslintrc.js
rules: {
'no-restricted-imports': ['error', {
patterns: [{
group: ['**/legacy-*', '**/*.legacy.*'],
message: 'Legacy APIs are deprecated. Check migration guide.'
}]
}]
}