proofreading
This commit is contained in:
+25
-25
@@ -1,44 +1,44 @@
|
||||
# Сдаём задачу спринта 26 на ревью — 3/3 эпика
|
||||
|
||||
> Срок проверки этого задания и доработок по нему составят не более 48 часов.
|
||||
> Срок проверки этого задания и доработок по нему составит не более 48 часов.
|
||||
|
||||
Вы завершаете финальный, 26-й спринт. Пришло время для ревью!
|
||||
|
||||
## В этом спринте ревью состоит из двух итераций:
|
||||
## В этом спринте ревью состоит из двух итераций
|
||||
|
||||
1. Вы сдаёте свой полностью готовый эпик. Ревьювер оценит работу по всем критериям из таблицы, оставит комментарии в вашем Pull Request. Вы вносите последние изменения по замечаниям ревьювера и получаете от него разрешение сливать работу в общую ветку. Эту итерацию вы сдаёте в окошко к этому уроку;
|
||||
2. Вы мерджите свой эпик с коллегами по команде в общую ветку. Как только закончили – присылаете Pull Request из общей ветки в master на финальное ревью! Эту итерацию вы сдаёте в окошко к следующему уроку.
|
||||
1. Вы сдаёте свой полностью готовый эпик. Ревьюер оценит работу по всем критериям из таблицы, оставит комментарии в вашем Pull Request. Вы вносите последние изменения по замечаниям ревьюера и получаете от него разрешение на слияние с общей веткой. Эту итерацию вы сдаёте в окошко к этому уроку.
|
||||
2. Вы мержите свой эпик с коллегами по команде в общую ветку. Как только закончили — присылаете Pull Request из общей ветки в `master` на финальное ревью! Эту итерацию вы сдаёте в окошко к следующему уроку.
|
||||
|
||||
> 📌 Не забывайте руководствоваться **правилами из темы спринта про Git** (урок 4 “Работа над проектом”).
|
||||
> 📌 Не забывайте руководствоваться **правилами из темы спринта про Git** (урок 4 «Работа над проектом»).
|
||||
|
||||
Если вы закончили работу над вашим эпиком и результат соответствует базовой версии критериев, **вы можете выбрать одну или несколько дополнительных задач** (согласовав с другими участниками команды). К таким задачам относятся:
|
||||
Если вы закончили работу над своим эпиком и результат соответствует базовой версии критериев, **вы можете выбрать одну или несколько дополнительных задач** (согласовав с другими участниками команды). К таким задачам относятся:
|
||||
|
||||
- локализация
|
||||
- тёмная тема
|
||||
- Яндекс Метрика
|
||||
- экран авторизации
|
||||
- экран онбординга
|
||||
- алерт с предложением оценить приложение
|
||||
- сообщение о сетевых ошибках
|
||||
- launch screen
|
||||
- поиск по таблице/коллекции в своём эпике
|
||||
- локализация,
|
||||
- тёмная тема,
|
||||
- Яндекс Метрика,
|
||||
- экран авторизации,
|
||||
- экран онбординга,
|
||||
- алерт с предложением оценить приложение,
|
||||
- сообщение о сетевых ошибках,
|
||||
- launch screen,
|
||||
- поиск по таблице/коллекции в своём эпике.
|
||||
|
||||
При работе над дополнительными задачами руководствуйтесь теми же рекомендациями, что и при работе над основными задачами.
|
||||
|
||||
Некоторые дополнительные задачи будет гораздо проще сделать, если при разработке эпика вы будете сразу писать код с их учетом. Для локализации – не объявлять видимые для пользователя строки в коде, а использовать NSLocalizedString. Для темной темы – задавать цвета не константами, а с использованием каталога ассетов. Даже если вы не будете делать дополнительные задачи, рекомендуем учитывать это при разработке эпиков – это хорошая привычка, которая делает вам более ценным разработчиком.
|
||||
Некоторые дополнительные задачи будет гораздо проще сделать, если при разработке эпика вы будете сразу писать код с их учётом. Для локализации — не объявлять видимые для пользователя строки в коде, а использовать `NSLocalizedString`. Для тёмной темы — задавать цвета не константами, а с использованием каталога ассетов. Даже если вы не будете делать дополнительные задачи, рекомендуем учитывать это при разработке эпиков — это хорошая привычка, которая делает вас более ценным разработчиком.
|
||||
|
||||
Если вы работали над дополнительными задачами, они также должны быть влиты в мастер-ветку вашей команды.
|
||||
|
||||
### Убедитесь, что для первой итерации ревью:
|
||||
## Убедитесь, что для первой итерации ревью
|
||||
|
||||
1. Вы полностью реализовали свой эпик;
|
||||
2. Вы обработали все замечания от коллег в команде и ревьювера;
|
||||
3. Вы сдаёте ссылку на Pull Request с вашим эпиком;
|
||||
4. Сделанная вами работа соответствует базовому варианту из таблицы с критериями;
|
||||
5. Вы исправили критические комментарии ревьювера;
|
||||
6. Вы получили разрешение от ревьювера сливать ветку с командой.
|
||||
1. Вы полностью реализовали свой эпик.
|
||||
2. Обработали все замечания от коллег в команде и ревьюера.
|
||||
3. Сдаёте ссылку на Pull Request с вашим эпиком.
|
||||
4. Сделанная вами работа соответствует базовому варианту из таблицы с критериями.
|
||||
5. Исправили критические комментарии ревьюера.
|
||||
6. Получили разрешение от ревьюера на слияние ветки с командной.
|
||||
|
||||
[Критерии ревью индивидуального PR если вы выбрали `UIKit`](https://code.s3.yandex.net/Mobile/iOS/pdf/...)
|
||||
[Критерии ревью индивидуального PR если вы выбрали `SwiftUI`](https://code.s3.yandex.net/Mobile/iOS/pdf/...)
|
||||
[Критерии ревью индивидуального PR, если вы выбрали UIKit.](https://code.s3.yandex.net/Mobile/iOS/pdf/...)
|
||||
[Критерии ревью индивидуального PR, если вы выбрали SwiftUI.](https://code.s3.yandex.net/Mobile/iOS/pdf/...)
|
||||
|
||||

|
||||
|
||||
Reference in New Issue
Block a user