在GitHub上,什么's审稿人和受让人之间的区别?

2016年12月7日在GitHub博客上宣布的新功能引入了选项向Pull Request添加审阅者

GitHub Review Option

您现在可以明确地向合作者请求审查,从而更容易指定您希望谁审查您的pull请求。

您还可以在拉请求页面侧栏中看到您正在等待审阅的人员的列表,以及已经离开审阅的人员的审阅状态。

但是,已经通过分配人员(因此可选项)为PR显式设置了审查员。

现在两个选项都有了,既然它们都有相同的最终目标,那么每个选项的作用是什么呢?

114408 次浏览

在GitHub中,评审员是审查拉请求的人。项目所有者可以向任何维护者请求审查,他们甚至可以设置一个选项,这样只有当其中一个具有写权限的维护者审查时,才可以合并pull请求。

根据官方github的文档,受让人是负责特定问题和拉请求的人。作为一名审稿人,它有时会感到困惑。它实际上是用来解决问题的,而不是拉请求,这样当我们收到一个问题时,我们就可以分配一个人来解决它。在拉取请求中,受让人指的是在收到其他维护者的评论和更改请求后,负责合并该拉取请求的人。

编辑:

在与几个OSS维护者讨论后,评论家被定义为这个词应该是:审查(某人的代码),而“受让人”的定义在下面解释。

“评论家”:你想要检查代码的人。不一定是负责该领域或合并提交的人。就像GitHub自动提示的那样,可能是以前处理过这段代码的人。

“受让人”:它的含义取决于项目团队/维护者,没有严格的定义。它可以是PR的开启者,也可以是负责该领域的人(他将在审查完成后接受PR,或者只是关闭它)。GitHub并不能定义它为项目维护者开放什么,什么最适合他们的项目。

之前的回答:

好吧,我继续回答我自己的问题。

对于有写权限用户的PR:,受让人将是打开PR的同一个人,而reviewer将取代旧的受让人函数(审查代码),成为受让人选择的人。

具有写权限的人将分配自己(或其他有写特权的成员)来审查PR(审稿人)。受让人为空白。

对于外部贡献者未完成的PR:写访问成员将接受未完成的工作并为她分配。她将负责完成任务,作为受让人。因为pr的主要目的是审核变更,所以她会选择其他一些人来审核变更。

根据接受的答案。是的,“受让人”的定义比较宽松,可以根据不同团队的需要进行不同的使用。

在我们由8名开发者组成的团队中,大多数PR都有1名审查员,他负责提出修改建议并最终批准PR。在审查员阶段,“受让人”是打开PR的人;之后,如果PR被其他开发者接手,就会增加一个新的“受让人”。一旦PR被批准并准备好进行QA或直接合并,就会添加一个新的QA“受让人”。这样“受让人”列表就会增加。

我们用“受让人”来统称下列人员:

  1. 拉请求作者
  2. 作者工作PR变更建议(通常为1)
  3. QA相关人员
  4. 负责合并的人员(通常为2或3人)

使用“受让人”便于将来定位PR。我的一个项目有>3000 pr。

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

或者只是author:raya-dumas来查找作者创建的所有项目(issue, pr)

和其他类似的查询,以简化搜索过程。“里程碑”对于简化PR搜索也很有帮助。

截图Github, Q4 2017

之前,GitHub只有一个受让人字段,没有审稿人字段。当时没有区别,所以assignee字段最常被用作审阅者字段。

但是要以适合你项目的方式使用它们。

“评论者”之间最大的区别是;和“;assignees"根据GitHub,审稿人实际上有一个跟踪状态——他们是否审查了PR ?

当你添加一个审阅者时,它实际做的是创建一个“检查request":GitHub review request pending

审稿人收到通知(就像一个"assign "would),但现在他们实际上有一个任务,他们可以完整的,这是在一个拉请求上提供一个“review": GitHub review changes dialog box . < / p >

在审阅者离开评审(批准或请求更改)后,该信息将在GitHub API和接口中跟踪:

GitHub reviewer approved

GitHub review required PR list

对于被委派的人,你可以将他们与PR联系起来,但除此之外,GitHub并不真正关心这意味着什么或这些人需要做什么。对于审阅者,你可以使用新的搜索查询“protect"分支机构,用CODEOWNERS手工或通过PullApprove等工具,围绕评审任务和工作流构建更深层次的API集成分配审阅者。

另一个区别是:创建PR的人可以将自己指定为受让人,但不能要求自己成为审稿人之一。