2016年12月7日在GitHub博客上宣布的新功能引入了选项向Pull Request添加审阅者
您现在可以明确地向合作者请求审查,从而更容易指定您希望谁审查您的pull请求。 您还可以在拉请求页面侧栏中看到您正在等待审阅的人员的列表,以及已经离开审阅的人员的审阅状态。
您现在可以明确地向合作者请求审查,从而更容易指定您希望谁审查您的pull请求。
您还可以在拉请求页面侧栏中看到您正在等待审阅的人员的列表,以及已经离开审阅的人员的审阅状态。
但是,已经通过分配人员(因此可选项)为PR显式设置了审查员。
现在两个选项都有了,既然它们都有相同的最终目标,那么每个选项的作用是什么呢?
在GitHub中,评审员是审查拉请求的人。项目所有者可以向任何维护者请求审查,他们甚至可以设置一个选项,这样只有当其中一个具有写权限的维护者审查时,才可以合并pull请求。
根据官方github的文档,受让人是负责特定问题和拉请求的人。作为一名审稿人,它有时会感到困惑。它实际上是用来解决问题的,而不是拉请求,这样当我们收到一个问题时,我们就可以分配一个人来解决它。在拉取请求中,受让人指的是在收到其他维护者的评论和更改请求后,负责合并该拉取请求的人。
编辑:
在与几个OSS维护者讨论后,评论家被定义为这个词应该是:审查(某人的代码),而“受让人”的定义在下面解释。
“评论家”:你想要检查代码的人。不一定是负责该领域或合并提交的人。就像GitHub自动提示的那样,可能是以前处理过这段代码的人。
“受让人”:它的含义取决于项目团队/维护者,没有严格的定义。它可以是PR的开启者,也可以是负责该领域的人(他将在审查完成后接受PR,或者只是关闭它)。GitHub并不能定义它为项目维护者开放什么,什么最适合他们的项目。
之前的回答:
好吧,我继续回答我自己的问题。
对于有写权限用户的PR:,受让人将是打开PR的同一个人,而reviewer将取代旧的受让人函数(审查代码),成为受让人选择的人。
具有写权限的人将分配自己(或其他有写特权的成员)来审查PR(审稿人)。受让人为空白。
对于外部贡献者未完成的PR:写访问成员将接受未完成的工作并为她分配。她将负责完成任务,作为受让人。因为pr的主要目的是审核变更,所以她会选择其他一些人来审核变更。
根据接受的答案。是的,“受让人”的定义比较宽松,可以根据不同团队的需要进行不同的使用。
在我们由8名开发者组成的团队中,大多数PR都有1名审查员,他负责提出修改建议并最终批准PR。在审查员阶段,“受让人”是打开PR的人;之后,如果PR被其他开发者接手,就会增加一个新的“受让人”。一旦PR被批准并准备好进行QA或直接合并,就会添加一个新的QA“受让人”。这样“受让人”列表就会增加。
我们用“受让人”来统称下列人员:
使用“受让人”便于将来定位PR。我的一个项目有>3000 pr。
is:open is:pr author:raya-dumas
is:closed is:pr assignee:raya-dumas
或者只是author:raya-dumas来查找作者创建的所有项目(issue, pr)
author:raya-dumas
和其他类似的查询,以简化搜索过程。“里程碑”对于简化PR搜索也很有帮助。
之前,GitHub只有一个受让人字段,没有审稿人字段。当时没有区别,所以assignee字段最常被用作审阅者字段。
但是要以适合你项目的方式使用它们。
“评论者”之间最大的区别是;和“;assignees"根据GitHub,审稿人实际上有一个跟踪状态——他们是否审查了PR ?
当你添加一个审阅者时,它实际做的是创建一个“检查request":
审稿人收到通知(就像一个"assign "would),但现在他们实际上有一个任务,他们可以完整的,这是在一个拉请求上提供一个“review": . < / p >
在审阅者离开评审(批准或请求更改)后,该信息将在GitHub API和接口中跟踪:
对于被委派的人,你可以将他们与PR联系起来,但除此之外,GitHub并不真正关心这意味着什么或这些人需要做什么。对于审阅者,你可以使用新的搜索查询, “protect"分支机构,用CODEOWNERS和手工或通过PullApprove等工具,围绕评审任务和工作流构建更深层次的API集成分配审阅者。
另一个区别是:创建PR的人可以将自己指定为受让人,但不能要求自己成为审稿人之一。