-
Notifications
You must be signed in to change notification settings - Fork 7
Dev: Pull Requests
taery edited this page Oct 14, 2016
·
3 revisions
Pull Request'ы (PR) позволяют всем участникам команды быть в курсе, что делают другие. В общем случае любая значимая часть кода появляется в ветке develop
только посредством PR.
В общем случае последовательность действий такая:
- создать ветку
feature/осмысленное-имя
(отdevelop
), - сделать туда кучу коммитов,
- создать PR (в
develop
), когда думаете что все готово (на самом деле это не так), - в описании указать ссылку на задачу из YouTrack (если она есть)
- добавить в PR сопроводительные картинки и анимации, если были изменения в UI
- @упомянуть с галочками в PR, тех кто по вашему мнению должен просмотреть нововведения,
- поставить себя в качестве
assignee
(себя и всех, кто еще будет коммитить в ветку вашего PR), - реагировать на комменты по PR, исправлять недочеты коммитами в ветку,
- убедиться что все одобрили коммит (это видно по галочкам) а все недочеты исправлены
- PROFIT.