Разработка через pull-requestы?

Question

[magary4]

сейчас в проекте есть человек который ревьювит пулл-реквесты от остальных 4 разработчиков и отправляет их на тестовый сервер

это все отлично когда разработчик внедряет новый функционал который затрагивает различные модули в разных частях проекта с которыми он не очень знаком. т.е может что-то сломать ненароком сам того не понимая

но что если если он добавляет один класс в css или 2 строчки в twig - темплейт или правит файл переводов?
опять надо создавать пулл-реквест и ждать пока некто соизволит обновить сервер

в моем понимании это сильно тормозит процесс и нужнен обходной путь без пулл-реквестов для простых правок
как?

Answer 1

[tema_sun]

Не нужен. Будет бардак.

Answer 2

[sim3x]

TDD

Answer 3

[Константин Китманов]

У вас, получается, бутылочное горлышко -- тот единственный человек, который имеет право это делать. А что если он заболеет, уволится, уедет на #сказочноебали -- работа встанет? Это неправильно.
Можно начать с того, что мелкие правки может ревьюить кто угодно, это как раз хороший способ въехать в проект. Большие правки можно ревьюить вдвоем. По мере роста квалификации и знания проекта каждый сможет делать хорошее ревью, это пойдет на пользу и проекту и разработчикам.

Comments

[Максим Мосейчук]

Дело в том, что репозитории не различают мелкие и крупные правки. А давать права в надежде, что разработчик будет принимать лишь мелкие правки - так себе затея.

[Константин Китманов]

fshp: а это не вы тот самый человек?:)

[Максим Мосейчук]

Константин Китманов: к сожалению, в моей команде я единственный, кто знает что такое пул реквест.

[Константин Китманов]

fshp: сочувствую.

[Anton]

fshp: легче тогда объяснить команде что это такое. Это же не слодная вещь

[Максим Мосейчук]

Anton: никому это не нужно. Вот прямо сейчас в чатике меня пытаются убедить, что свн лучше.

[Максим Мосейчук]

Anton: что б вы понимали, я уже скоро год как принимаю правки от начальника по скайпу. Он мне скидывает файл, я коммичу в репозиторий.

[magary4]

ну у меня ситуация немного другая
разработчиков не много и со всеми можно договорится
сказать "если ты правишь цсс или шаблон пуш делай в ветку melkie-pravki" для остальных вещей - новая ветка и пуллреквест

вот счас родилась идея сделать хук на пуш в ветку melkie-pravki который сразу сделает на превью-сервере git merge веток melkie-pravki & master

[magary4]

magary4: т.е все разработчики пушат в мелкие правки цсс в melkie-pravki и переодически это все мержится "главным" в мастер и пушится
но на превью сервер попадает немедленно

[Максим Мосейчук]

magary4: ну примерно так и устроена разработка linux

Answer 4

[Станислав Почепко]

Обходной путь - и тогда смысел от ревью? ЧТо бы ещё хуже что-то кто-то сделал?