Анкета/контрольный список для рецензентов Bitbucket Pullrequest

Я пытаюсь выяснить, смогу ли я ввести анкету в Bitbucket до того, как рецензенты одобрят запрос на слияние. Примеры вопросов, которые я хотел бы использовать:

  1. Если внесенные изменения относятся к глобальному репозиторию, совместно используемому многими регионами, уточнили ли вы у рецензентов, сообщил ли он/она глобальной команде или каким-либо конкретным лицам?
  2. Вы проверяли соглашения об именах переменных?
  3. Достаточно ли комментариев вокруг любой сложной логики?
  4. Достаточно ли модульных тестов вокруг новой логики, и где это уместно, проверяли ли вы отчет о покрытии кода?

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

Вопрос: Кто-нибудь задумывался о подобной ситуации и нашел какую-либо поддержку непосредственно в Bitbucket или с помощью каких-либо плагинов? Мой быстрый поиск в Google не дал никаких полезных результатов


git bitbucket pull-request bitbucket-server
person endless    schedule 17.07.2016    source источник

Ответы (4)


arrow_upward
4
arrow_downward

надстройка Scriptrunner сообщает, что может создавать задачи в запросе на вытягивание при создании запроса на вытягивание.

Вы также можете посмотреть и проголосовать за предложение по добавлению поведения в качестве основного на jira.atlassian.com< /а>.

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

Раскрытие информации: я работаю в Atlassian

person Rog    schedule 19.07.2016

arrow_upward
4
arrow_downward

Проверьте плагин «Руководство по запросам на слияние»:

https://prguidelines.services.atlassian.com/

person ykaganovich    schedule 24.08.2016

arrow_upward
1
arrow_downward

Если вы ищете способ иметь простые настраиваемые контрольные списки (задачи) в ваших запросах на вытягивание, похоже, это функция Премиум-версии Bitbucket, как описано в этом сообщение в блоге bitbucket, вы можете создавать задачи из комментариев (любая версия битбакет):

нажмите Создать задачу из комментария

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

Использование задач dedfault для bitbucket app вы можете настроить задачи по умолчанию для каждого репозитория. например если у вас есть репозиторий, в котором много изменений схемы, вы можете всегда включать задачу "Проверили ли вы с группой администраторов баз данных, влияет ли эта схема на последующие процессы" или "Влияет ли код соответствует стандартам и соглашениям компании по кодированию".

person alexkb    schedule 21.06.2018

arrow_upward
0
arrow_downward

У меня нет решений для всех ваших запросов, но есть как минимум 2 из них:

Проверили ли вы соглашения об именах переменных? Это должно быть сделано автоматически еще до создания PR. Вы можете использовать checkstyle/findbugs или любой другой инструмент статического анализа. Пусть люди сосредоточатся на просмотре фактических изменений, а не на названии.

Достаточно ли модульных тестов вокруг новой логики, и там, где это уместно, вы проверяли отчет о покрытии кода? Единственное, что я узнал, — люди никогда не проверяют отчет о покрытии (или любой другой). Так что вместо этого принесите эту информацию о покрытии в PR. Существует плагин покрытия кода. для Bitbucket Server, который вы могли бы использовать.

person artkoshelev    schedule 30.04.2018