Привет! Это чеклист человека создающего PR.
В зависимости от типа задачи которую ты выполнил (верстка или логика) оставь только ту часть чеклиста, которая соответствует твоей задаче.
Удали ту часть что не соответствует твоей задаче (для особо одаренных - Общее
удалять нельзя). А потом удали все до # Чеклист
В оставшиеся пункты проставь галки и заполни оставшуюся часть шаблона.
Удачи!
Общее:
- Я запустил проект перед тем как создать PR
- Я запустил тесты перед тем как создать PR
Верстка:
- Я проверил, что стили и отступы соответствуют дизайну
- Я проверил верстку на iPhoneSE, iPhone 8+, iPhone 11 Pro Max
- Я проверил верстку на iOS 12, 13
- Если на экране есть сворачивающиеся ячейки - проверил что нет крашей при сворачивании/разворачивании
- Если на экране есть текстовые поля - проверил что самое нижнее поле ввода никогда не прекрывается клавиатурой
Логика:
- Я проверил все кейсы из ТЗ как минимум через Mocker
- Я проверил, что сетевой слой работает правильно
Сворачивающиеся ячейки:
- Если на экране есть сворачивающиеся ячейки проверил что при сворачивании/разворачивании нет крашей
- Если на экране есть сворачивающиеся ячейки проверил что не ломается стейт и логика работает верно
Напиши кратко что сделал:
- Первое
- Второе
- Третье
- ...
- На первое
- На второе
- На третье
- ...
Чеклист для UI Specialist
- Все сверстано точно по макетам
- Есть пресс-стейты
- Лоад-стейты и еррор-стейты отображаются правильно
- Навигация осуществляется правильно (стэк/модалка) и презентация прваильная
- Верстка соответствет требованиям (все элеемнты есть, все цвета верные)
- Не добавлено лишних стилей
- Все цвета используются из палитры
- Поддерживается конвенция именования стилей и ресурсов
Чеклист для Logic Specialist
- Не нарушены ответственности
- Выполнены требования из ТЗ
- Для сложных (код который не понятен за 7 секунд или у которого много сайд-эффектов) мест написаны тесты
- В сложных местах есть комментарии
- Соблюдается оговоренной архитектурный подход