Як робити експертні огляди на запити на тягнення до GitHub?


12

Ми переходимо від Bitbucket до GitHub, і одна річ, з якою ми боремося, - це рецензування рівних кодів, які дуже плавно працювали на Bitbucket, як це:

  1. Автор відкрив запит на виклик (GitHub: те саме)
  2. Автор додав своїх колег як рецензентів (GitHub: тут бореться з кількома привласниками)
  3. Рецензент:
    1. Затверджено PR із зеленою галочкою (GitHub: ??)
    2. Додані коментарі (GitHub: те саме)
    3. Створено полегшені завдання (GitHub: подібне, якщо - [ ]в описі PR використовується синтаксис; шкода, що він не працює для завдань)
  4. Існує список PR-сайтів, де я з першого погляду бачу переглянуті і добре об'єднані і потребують подальшої уваги (GitHub: ??)

Я мушу зазначити, що ми хочемо уникнути сторонніх інструментів перегляду коду, якщо це можливо, і хотіли б залишитися на ванільному GitHub з деякими шляхами вирішення.


1
Здається, що ви можете переключитися передчасно. Навіщо все-таки перемикатися, особливо якщо нова річ не має всіх необхідних вам функцій?
няня

Напишіть коментар до prq та виділіть @ хто хоче отримати сповіщення. Рецензент може додавати теги, щоб показати свою думку відгуків.
Вільберт

Відповіді:


6

З того, що я бачив, більшість цих кроків робиться на Github за домовленістю, а не будь-яким офіційним процесом, наданим Github.

Мій роботодавець використовує Github, я керую великою кількістю невеликих проектів з відкритим кодом, а також беру участь у інших проектах з відкритим кодом.

Ось як я зазвичай бачив це зроблено:

Автор додав своїх рецензентів своїх колег:

Це залежить від проекту до проекту, але в цілому призначені експертні рецензенти є всіма учасниками проекту .

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

У магазині, де я зараз працюю, ми зливаємося після того, як хтось із півдесятка розробників у команді дав згоду.

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

Затвердження рецензента:

Затвердження, як правило, демонструється, коментуючи запит на виклик із зазначенням "+1" або "lgtm" (мені добре виглядає).

Легкі завдання:

Я також використовував прапорці, але в більшості випадків кожен коментар до запиту на виклик вважається неявним "завданням", яке вирішується або:

  • зміна коду, який коментує рядок
  • відповівши іншим коментарем

З першого погляду побачивши, що схвалено та що ще потрібно переглянути:

Я використовував розширення Looks Good To Me для Chrome, яке надає вам такий вид з екрана "Затягнути запити". Здається, подання списку запитів на виклик було порушено останніми змінами в Github.

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