Bitbucket拉取请求审核者问卷/清单

12

我正在尝试在Bitbucket上强制要求审核者在批准拉取请求之前填写问卷。以下是一些我想使用的样本问题:

  1. 如果所做的更改是针对许多地区共享的全局存储库,请确认是否与审阅者核实是否已通知了全球团队或任何特定个人?
  2. 检查了变量命名约定吗?
  3. 是否在任何复杂逻辑周围有足够的注释?
  4. 新逻辑周围是否有足够的单元测试,而且在相关的情况下,是否检查了代码覆盖率报告?

这些只是一些示例问题,其中有些当然可以在构建过程中或在Sonar等工具中使用各种插件进行监控,但总有一些无法立即自动化的事情,例如上面的第一个示例问题。因此,在它们可以自动化之前,我至少想确保在批准拉取请求之前回答这些问题。

问题:是否有人考虑过类似的情况,并在Bitbucket内部或使用任何插件找到支持?我的快速谷歌搜索没有产生有用的结果。

4个回答

5

编辑: 正如正确指出的那样,此功能已被停用

我们鼓励团队使用任务来执行任何他们想要强制执行的指南,并鼓励团队继续使用CONTRIBUTING.md文件作为贡献指南。


过时的答案:

请查看Pull Request Guidelines插件:

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


这是一个失效的链接 - Atlassian已经停用了BB PR指南。请参阅https://community.atlassian.com/t5/Bitbucket-articles/PR-Guidelines-Functionality-Discontinuation-Notice/ba-p/1700276 - stwr667

4
Scriptrunner 插件 可以在创建 pull request 时创建任务。

您也可以在jira.atlassian.com上关注并投票支持将该功能作为核心添加的建议。

尽管如此,这是一种有些不寻常的请求,因为它可能会增加过程开销。您列出的任务应该在创建 pull request 前(希望当然)进行检查,并且审核人员应在其审核中标记缺失的部分。

披露:我为 Atlassian 工作。


2
如果您正在寻找一种在拉取请求中拥有简单自定义检查清单(任务)的方法,您可以从评论中创建任务(任何版本的Bitbucket)。请注意,Bitbucket的高级版似乎也支持此功能,详情请参阅Bitbucket博客文章。请保留HTML标签。

click Create task from a comment

然而,如果您想要能够“要求拉取请求任务完成”,则需要使用“高级”计划。使用Bitbucket默认任务应用程序,您可以在每个仓库基础上设置默认任务。例如,如果您有一个重度架构更改的存储库,您可能希望始终包括以下任务:“是否已经与DBA团队进行了检查以确定此架构是否影响下游进程”,或者“代码是否符合公司编码标准和约定”。

1

我没有所有请求的解决方案,但至少有两个:

您是否检查了变量命名约定? 这应该在创建PR之前自动完成。 您可以使用checkstyle / findbugs或任何其他linting /静态分析工具。 让人们集中精力审查实际更改而不是命名。

新逻辑周围是否有足够的单元测试,并且您是否检查了代码覆盖率报告? 我学到的一件事-人们从不检查覆盖范围(或任何其他)报告。 因此,将覆盖信息带入PR中。 Bitbucket Server有一个{{link1:Code Coverage Plugin}}可供使用。


网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接