Представляем Pull Requests для команды из 2 человек - объединить мои собственные запросы?


11

Я представляю git младшему члену команды (кооператив).

Теперь они знакомы с основами добавления, фиксации, толкания и вытягивания.

Теперь я хочу представить их для получения запросов и веток.

Если они начнут делать запросы извлечения в филиалах, я должен сделать то же самое для моей текущей работы?
Я буду тем, кто объединяет их запросы на получение. Я не был уверен, будет ли для меня больше смысла работать в филиалах (в общем, это хорошая практика, я знаю, но мне любопытно, что это специфическая ситуация с двумя разработчиками и одним младшим). ) и если это так, это означает, что я просто буду объединять свои собственные ветви в master. Буду ли я в любом случае делать запрос на получение доступа к моей работе / филиалам? Обычно для этих изменений мы используем базовый рабочий процесс ветки github:
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

Есть ли цель использовать запросы на извлечение в моем репо, если я единственный разработчик? полезно, но не совсем так конкретно.

Какой рабочий процесс с двумя людьми над проектом также кажется более общим

а также

Должен ли я открывать пул-запросы из ветки официального репо или моего форка? кажется больше о вилках.

Ответы:


19

Нет. Вы не должны объединять свои собственные запросы. Что хорошо для гуся, то хорошо для гусенка. Объединение ваших собственных запросов на получение доступа создает плохой прецедент для нашего младшего разработчика. Это также означает, что никто не смотрит на ваш код. Независимо от того, насколько мы старшие, мы все делаем ошибки и время от времени пишем плохой код. Научите своего младшего, как работают обзоры кода, с другой стороны, заставляя его проверять и объединять вашу работу.

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

  1. Вы оба узнаете, где ваш младший должен сосредоточиться на учебной деятельности.
  2. Вы узнаете, где вы умнее, чем должны быть.

6
Другим огромным преимуществом обзоров кода является то, что по крайней мере два человека видели, знали и имели возможность задавать вопросы о каждом отдельном изменении кода, прежде чем оно вступит в силу. Даже если младший разработчик не знает, что искать, он гарантированно научиться чему-нибудь из всего этого.
Ixrec
Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.