ИС
Как говорили в одной моей бывшей команде — о, ещё один замьюченый чатик!
Size: a a a
2021 April 06
RU
Хах
RU
неужели нет никакого боевого устава написанного кровью про эту самую культуру? гит флоу может, в эту сторону копать?
O
Я вот не понимаю как пользоваться мерж реквестами. У нас в команде ревью делается в фишае, хотели перейти на гит, но не поняли как обойти ограничения.
Как быть, если я хочу добавить в ревью отфильтрованные по какому-то признаку файлы? Там действительно может быть лишнее.
Как быть, если смержить надо сейчас, а проревьюить можно и потом?
Как быть, если я хочу добавить в ревью отфильтрованные по какому-то признаку файлы? Там действительно может быть лишнее.
Как быть, если смержить надо сейчас, а проревьюить можно и потом?
O
И какой смысл запрещать кому-то мерджить (если нет обхективных на то причин)?
Если ревьюер захочет посмотреть - он и без этого посмотрит. Если не захочет, то просто на кнопку нажмет, только придется его заставлять эту кнопку нажать. Какое-то бестолковое ограничение
Если ревьюер захочет посмотреть - он и без этого посмотрит. Если не захочет, то просто на кнопку нажмет, только придется его заставлять эту кнопку нажать. Какое-то бестолковое ограничение
O
Я так понял, что это флоу придумано для опен сорса, когда много человек коммитят в репозиторий и их обязательно надо проревтюить
ЕГ
Звучит будто вы наставили себе костылей раньше, а теперь ищете костыли, которые позволят вам обойти ограничения)
Зачем вам мержить прямо сейчас, вот прям срочно-срочно, что не может подождать?
Зачем вам мержить прямо сейчас, вот прям срочно-срочно, что не может подождать?
O
А когда это один человек коммитит в свой собственный свои автотесты, какой резон ему запрещать это делать?
ИС
>
Как быть, если смержить надо сейчас, а проревьюить можно и потом?
Никак. Так быть не должно. Если вам кажется, что у вас так работает, и ничего страшного в этом нет, то это ошибка.ИС
Причём, где-то в корне процессов, если оказывается, что надо срочно мержить то, что ещё даже не готово.
O
Хорошо, по другому. Сейчас проревьюили и смержили. НО потом другой человек захочет посмотреть ревью, а оно исчезло вместе в МР
ЕГ
МРы не исчезают. Нет проблем зайти в закрытые и открыть нужный
EB
В гитхабе никуда pulled requests не пропадают. На них вешается тэг «закрытый»
EB
Ссылка на PR будет валидная и после того, как его смерджили, а ветку удалили
O
хм, проглядел значит это
O
а как отфильтровать?
ЕГ
Запушить только то, что нужно, сделать МР, не удалять ветку, продолжить работать в ней потом. Не катит как вариант? Не очень понял просто что именно хочется)
O
Есть большая фича, под нее есть ветка. Отдельные разработчики делают ее части, могут отрезать свои ветки под них. Но в результате в дев бранчу должна пойти фича целиком. Как сделать, что б каждый делал ревью под свою часть отдельно ?
ИС
Сначала разбить задачу на подзадачи так, чтобы у каждого разработчика была своя таска, и он в ней был один. И ветка была для этой таски отдельная. Затем открывать отдельные MR'ы в feature-ветку.
ИС
И стараться декомпозировать и делить в команде задачи так, чтобы не было такого, что "код этого разработчика нужен в задаче вот того, ещё до тех пор, пока он будет вмержен".