Перейти к основному содержимому

Процедуры контроля и автоматизация

Автор 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.'
}]
}]
}