У GitHub, яка різниця між рецензентами та правонаступниками?


187

Функція, додана 7 грудня 2016 року, анонсована в блозі GitHub, запровадила можливість додавання рецензентів до запиту на видалення

Варіант огляду GitHub

Тепер ви можете надіслати запит на рецензію прямо від співпрацівників, що спрощує вказівку, на кого ви хочете переглянути ваш запит на виклик.

На бічній панелі сторінки запиту на запит ви також можете побачити список людей, яких ви очікуєте на огляд, а також статус відгуків тих, хто вже їх покинув.

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

З якими зараз доступними обидва варіанти, яка роль кожного з варіантів, оскільки вони мають однакову кінцеву мету?


1
коли "особливість правонаступника" випускається вперше? Чи є якась стаття, яка її вводить?
бабей

Відповіді:


135

Редагувати:

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

Для "рецензента" : хтось, який ви хочете переглянути код. Не обов'язково особа, відповідальна за цю область або відповідальна за об'єднання комітету. Може бути хтось, хто раніше працював над цим фрагментом коду, як автоматично пропонує GitHub.

Для "правонаступника" : до команди / керівника проекту, що це означає, і немає чіткого визначення. Це може бути відкривальник PR або хтось, відповідальний за цю сферу (хто збирається прийняти PR після завершення огляду або просто закрити його). GitHub не визначає, що залишає відкритим для керівників проектів, що найкраще підходить для їх проекту.

Попередня відповідь:

Гаразд, я продовжу відповідати на власне запитання.

Для PR користувачів з доступом до запису: Правоохоронник буде тим самим, хто відкрив PR, а рецензент замінить стару функцію правонаступника (рецензуючий код), будучи цим самим особою на вибір.

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

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


24
Кожному новому члену команди я повинен надіслати посилання на цю відповідь, щоб пояснити, як поводитися з привласниками та рецензентами. Що спонукає мене до думки, що тут щось принципово не так :(
Андрій Кулешов

Чи повинен правонаступник мати доступ для запису?
Emre Sülün

Чи є різниця в поведінці сповіщень електронною поштою між ними?
jxramos

26

У GitHub рецензент - це людина, яка переглядає запит на тягу. Власник проекту може подати запит на перегляд у будь-якого з обслуговуючого персоналу. Вони навіть можуть встановити опцію, щоб запит на витяг можна було об'єднати, лише якщо його перегляне один з обслуговуючого персоналу з доступом до запису.

Згідно з офіційною документацією github , Асигнант - це особа, яка працює над конкретними питаннями та тягне запити. Іноді його плутають як рецензента. Це насправді призначене для використання з питаннями, а не з запитом на виклик, так що коли ми отримуємо проблему, ми можемо призначити когось, щоб виправити це. У запиті на виклик правонаступник посилається на особу, яка відповідає за об'єднання цього запиту після отримання коментарів та зміни запитів інших обслуговуючих служб.


2
Дякую за відповідь, але я не думаю, що це питання вирішує повною мірою. Ви можете призначити проблему комусь (тож вона буде правонаступником), але коли PR буде надісланий, хтось буде рецензент (PR-одержувач), і в цей момент мені все ще не зрозуміло, чим відрізняється правонаступник від рецензент.
Цезар Аугусто

14

Відповідно до прийнятої відповіді. Так, "правонаступник" має більш низьке визначення і його можна використовувати по-різному відповідно до потреб команди.

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

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

  1. Витягніть запит автора
  2. Автор, який працює над пропозиціями щодо зміни PR (зазвичай такий самий, як 1)
  3. Залучена особа із забезпечення якості
  4. Особа, відповідальна за злиття (як правило, 2 або 3)

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

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Або просто author:raya-dumasзнайти всі створені автором елементи (випуски, PR)

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

Знімок екрана Github, 4 квартал 2017 року


Дуже добре пояснено.
Нітін Гаур

Слід зазначити, що ви можете просто шукати автора: my-github-handle, щоб знайти, що PR створила людина
Wisienkas

1

До GitHub було лише поле правонаступника та поле для рецензентів. Тоді не було різниці, тому поле правонаступника найчастіше використовується як поле рецензента.

Але використовуйте їх будь-яким способом, який підходить вашому проекту.

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.