Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added paragraph - short summary of development branch #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added paragraph - short summary of development branch #10

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b500f41
Update development-guidelines.md
tomatolog Nov 10, 2016
be70704
Update development-guidelines.md
tomatolog Nov 10, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions development-guidelines.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -587,6 +587,24 @@ patch series на ревью кода лидом проекта. Требова

Такая последовательность требуется, чтобы можно было пользоваться
`git bisect`.

## TLDR действия после bugfix в рабочей ветке

Работа ведется в 3х ветких master, dev и рабочая. Причем master!=dev и в мастер нельзя лить комиты, только ветки. Подробнее вро работу с ветками можно прочесть [Ветки](#Ветки) Имена master и dev уточняйте.


рабочая ветка никому кроме разработчика не нужна, для ревью следует создать ветку вида `<developer-initials>/<ticket_number>/<short_summary>`
Ветка для ревью должна быть создана от dev ветки, для того чтобы ревьюер \ тестер могли сразу проверить изменения.

При отправки кода на ревью, нужно:
* выбрать в ветку на ревью - только "нужные" изменения через merge+squash ветки разработчика, если изменения логически соответвуют одому комиту или же через cherry pick комитов из ветки разработчика, при этом сами комиты должны быть оформленны и содержать сообщения согласно [Коммиты](#Коммиты)
* запушить ветку для ревью
* отписать в тикет в Redmine о ветке, перевести Assignee на ревьюера, сменить статус тикета (On Blocking/Nonblocking Review)


После ревью, тикет Redmine вернется разработчику обратно:
* если необходимы дальнейшие исправления - то код, для повторной отправки на ревью, должен заливатся в новую ветку с суффиксом итерации `...._v<iteration_number>`. Как например - начальная ветка для ревью `dev1/7/fix_of_all_around`, а ветка с дальнейшими исправлениями будет названа как `dev1/7/fix_of_all_around_v2`
* если изменения должны быть опубликованы, то необходимио создать ветку уже от master, с таким же именем и суфиксом новой итерации. В эту ветку сделать rebase своих изменений из ветки одобренной на ревью. Можно использовать команду ```git rebase --onto master_v1 dev_v3 dev1/7/fix_of_all_around_v3```. После чего запушить ветку в мастер

# Про комментарии в коде (TODO)

Expand Down