BACK-11068: Правки security и встроенный постгрес в тестах
О деталях реализации этой задачи и сопутствующего ей процесса разработки можно рассказать здесь.
Памятка разработчика
Я убедился, что основные ветвления бизнес-логики
-
Логируют ключевые аргументы и переменные через
log.debug(...), или таких в этом MR нет. - Покрыты e2e-тестами, или таких в этом MR нет.
Публичный API для МП и Фронта
В текущей реализации добавлены, изменены или удалены публичные методы API?
- Да, и изменения отражены в документации API для МП.
- Нет / не требуется.
Схема хранения данных
В текущей реализации добавлены, изменены или удалены какие-либо колонки или таблицы?
- Да, и изменения отражены в документации Схемы данных.
- Нет / не требуется.
Дополнительные мелочи
- Сообщения коммитов соответствуют формату и оперируют бизнес-сущностями, а не техническими терминами.
- Создав этот MR, я открыл вкладку с изменениями и провёл самостоятельное ревью (self-pre-review).
- Задача на доске переведена в статус "Для ревью".
- В задаче на доске добавлены заметки для команды QA (куда обратить внимание при тестировании).
Памятка ревьюера
- Я назначен ревьюером в этом MR.
- Я назначен ревьюером в задаче на доске.
- Задача переведена в статус "Идёт ревью".