Как делать рецензии на запросы GitHub?


12

Мы переходим от Bitbucket к GitHub, и мы сталкиваемся с одной проблемой, с которой мы сталкиваемся с рецензиями кода, которые работали на Bitbucket очень гладко, например:

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

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


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

Напишите комментарий к своему prq и выделите @, если вы хотите получить уведомление. Рецензент может добавить теги, чтобы показать свое мнение.
Уилберт

Также обратите внимание, что вы можете @mention целую команду, например, @ MyCompany / developers
Whymarrh

Ответы:


6

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

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

Вот как я обычно это делал:

Автор добавляет своих коллег в качестве рецензентов:

Это варьируется от проекта к проекту, но в целом все назначенные коллеги-рецензенты являются участниками проекта .

Проекты с открытым исходным кодом, похоже, имеют грубую иерархию - возможно, их условием было бы объединение только после того, как «основной» участник дал согласие.

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

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

Утверждение рецензента:

Утверждение, как правило, показывается, когда комментирует запрос на получение ответа «+1» или «lgtm» (выглядит хорошо для меня).

Легкие задачи:

Я также использовал флажки, но в большинстве случаев каждый комментарий по запросу извлечения считается неявной «задачей», которая решается либо:

  • изменение кода, который комментирует строка
  • отвечая другим комментарием

Краткий обзор того, что одобрено, а что еще необходимо пересмотреть:

Я использовал расширение Looks Good To Me для Chrome, которое дает вам такой вид на экране «Запросы на извлечение». Тем не менее, представление списка запросов по запросу, похоже, было нарушено недавними изменениями в Github.

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