Критерии готовности задачи/фичи перед merge в main.
Обязательно
- Код реализует описанные в задаче требования и не добавляет неоговоренной функциональности.
- Код соответствует Coding Standards; проходят ruff, mypy и pre-commit.
- Добавлены или обновлены тесты, покрывающие новую/измененную логику; тесты проходят локально и в CI.
- Нет закомментированного «мертвого» кода и отладочных выводов.
- Merge Request заполнен: описание изменений, как проверить; указаны ревьюеры.
- Code review пройден; замечания учтены или согласованы.
- CI (build, test, lint) успешен для ветки.
При необходимости
- Обновлена документация (README, docs), если меняются контракты, конфигурация или поведение, важное для других разработчиков или эксплуатации.
- Миграции БД добавлены и протестированы; обратная совместимость учтена.
- Изменения API (proto/REST) согласованы и версионированы при несовместимости.
Не входит в DoD по умолчанию
- Отдельный релиз и деплой (выполняются по процессу релизов).
- Обновление внешней документации продукта (если не было явной части задачи).
Связанные страницы
- Code Review Guide — что проверять в ревью
- Git Workflow — процесс MR
- Release Process — выход в релиз