Skip to content
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 не надо, дабы при мерже не решать лишних конфликтов и не переписывать дату патча