-
Notifications
You must be signed in to change notification settings - Fork 52
Ru:pull request
AlexeyDsov edited this page Nov 5, 2011
·
2 revisions
- Баги, фичи и любые иземения желательно выполнять через pull request.
- Pull Request должен заводиться на мастер версию и в первую очередь мержиться именно мастер. Далее, особенно если это багфикс, мержить в остальные ветки (сейчас это 1.0)
- И бага и фича должны покрываться тестами и при этом ни один из тестов не должен быть поломан.
- Если ломается BC (Back Compatability) или прекращается поддержка какой либо старой версии автору необходимо это указывать и предупреждать, что в свою очередь требует дополнительного обсуждения.
- Мерж в мастер делается если контрибьютеры не против данного изменения и обсуждение окончено. Если Pull Request создал один из контрибьютеров, то ему в конечном итоге и мержить изменение.
- После выполнения мержа контрибьютер сделавший мерж добавляет запись об измениях в doc/ChangeLog с указанием даты мержа и автор(а|ов) патча. Заранее делать изменения doc/ChangeLog не надо, дабы при мерже не решать лишних конфликтов и не переписывать дату патча