На GitHub, какая разница между рецензентом и правопреемником?

Функция, добавленная 7 декабря 2016 года, анонсированная в блоге GitHub, внесла опцию чтобы добавить рецензентов в запрос Pull

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

Вы также можете увидеть список людей, которых вы ожидаете от просмотра на боковой панели страницы запроса запроса, а также статус отзывов от тех, кто уже покинул их.

Однако явная настройка рецензента для PR уже была выполнена путем назначения людей (вариант назначаемых лиц).

Теперь, когда доступны оба варианта, какова роль каждого параметра, поскольку оба они имеют одну и ту же конечную цель?

Ответ 1

EDIT:

После обсуждения с несколькими поддерживающими OSS рецензентов определяется как то, что должно было бы быть: просмотр (код кого-то) и "цессионарий" имеет более четкое определение, объясненное ниже.

Для "рецензента" : кого-то, кого вы хотите просмотреть. Не обязательно ответственное лицо за эту область или ответственное за слияние фиксации. Может быть, кто-то работал над этим куском кода раньше, как предлагает GitHub.

Для "цессионария" : до проектной команды/сопровождающего, что это значит, и нет строгого определения. Это может быть открывающий PR, или кто-то, кто отвечает за эту область (кто собирается принять PR после того, как обзор будет сделан или просто закроет его). Это не до GitHub, чтобы определить, что он оставляет для разработчиков проектов, что подходит для их проекта.

Предыдущий ответ:

Хорошо, я отвечу на свой вопрос.

Для PR пользователей с доступом для записи: Assignee будет тем же самым человеком, который открыл PR, и рецензент заменит бывшую функцию цессионария (обзор кода), являясь этим кем-то из правопреемников выбор.

Для пользователей PR без доступа к записи (сторонние участники): Кто-то, у кого есть права на запись, назначит себя (или другого члена, назначающего запись), для рассмотрения PR (Рецензента). Претендент пуст.

Для незавершенного PR от сторонних участников: участник записи права доступа выполнил незавершенную работу и назначил ей. Она будет отвечать за выполнение задания, будучи Assignee. Поскольку основная причина PR - это пересмотр изменений, она выберет некоторых других людей для пересмотра изменений.

Ответ 2

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

Согласно официальной документации github, Assignee - это человек, который работает над конкретными проблемами и запрашивает запросы. Иногда это путают как рецензент. На самом деле он предназначен для использования с проблемами, а не с запросом о выдаче, чтобы при получении проблемы мы могли назначить кого-то для ее устранения. В запросе на получение ответа правопреемник относится к лицу, отвечающему за объединение этого запроса после получения комментариев и запросов на изменение от других сопровождающих.

Ответ 3

Согласно принятому ответу. Да, "правопреемник" имеет более слабое определение и может использоваться по-разному для удовлетворения потребностей команды.

В нашей команде из 8 разработчиков, в большинстве PR у нас есть 1 рецензент, который предлагает изменения и в конечном итоге одобряет PR. На этапе проверки "цессионарий" - это лицо, открывшее PR; позже, если PR подхватит другой разработчик, будет добавлен новый "правопреемник". Как только PR одобрен и готов к QA или прямому слиянию, добавляется новый "уполномоченный" QA. Таким образом, список "правопреемников" растет.

Мы используем "правопреемник" для совместного обозначения следующих людей:

Автор запроса на тягу
Автор работает над предложениями по смене PR (обычно такой же, как 1)
QA человек вовлечен
Лицо, ответственное за слияние (обычно такое же, как 2 или 3)

Использование "правопреемника" помогает легко найти PR в будущем. Один из моих проектов имеет> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Или просто author:raya-dumas чтобы найти все элементы, созданные автором (проблемы, PR)

и другие подобные запросы, чтобы облегчить процесс поиска. "Вехи" также очень полезны для облегчения поиска в PR.