Ru:pull request

Jump to bottom

AlexeyDsov edited this page Nov 5, 2011 · 2 revisions

Общие правила заведения и рассмотрения Pull Request'ов и процесса разработки

Баги, фичи и любые иземения желательно выполнять через pull request.
Pull Request должен заводиться на мастер версию и в первую очередь мержиться именно мастер. Далее, особенно если это багфикс, мержить в остальные ветки (сейчас это 1.0)
И бага и фича должны покрываться тестами и при этом ни один из тестов не должен быть поломан.
Если ломается BC (Back Compatability) или прекращается поддержка какой либо старой версии автору необходимо это указывать и предупреждать, что в свою очередь требует дополнительного обсуждения.
Мерж в мастер делается если контрибьютеры не против данного изменения и обсуждение окончено. Если Pull Request создал один из контрибьютеров, то ему в конечном итоге и мержить изменение.
После выполнения мержа контрибьютер сделавший мерж добавляет запись об измениях в doc/ChangeLog с указанием даты мержа и автор(а|ов) патча. Заранее делать изменения doc/ChangeLog не надо, дабы при мерже не решать лишних конфликтов и не переписывать дату патча