修复某些问题后,我们应该在TFS中使用“拒绝”还是“等待作者”吗?


12

在TFS中,当我们在接受请求之前在“拉取请求”中对要修复的问题进行注释时,是否应将其标记为“拒绝”或“等待作者”?哪个更好?

在此处输入图片说明

Answers:


14

根据Microsoft在带有请求请求的Review代码上:对更改进行投票,建议每个批准类别的目的是:

同意建议:同意请求请求,但提供可选建议以改进代码。

等待作者:请勿批准更改,并请作者查看您的评论。作者解决了您的问题后,应该让您知道何时应该重新审阅代码。

拒绝:更改是不可接受的。如果您以这种方式投票,则应在拉取请求中留下评论,详细说明为何拒绝更改。

因此,我要说的Waiting for Author是,您认为作者认为他/她对解决方案的态度搞砸了,但是如果他们将您的评论放在心上,则他/她的代码是可以兑换的。

并且Rejected意味着无论代码编写得多么好,您都不会接受这种更改。

您必须问自己的问题是,您的小组对建议目的的想法是否与Microsoft的想法相符。


1
关于此内容:“在他们解决了您的疑虑之后,作者应该让您知道何时应该重新审阅代码。” -作者是否有一种机制可以提醒审阅者PR已准备好进行重新审阅?
罗伯特·辛

@RobertSim-我们使用的是“审阅者”下的“ Ping”选项,以发送消息以要求重新审阅。
killercowuk

看待此问题的另一种方式:Waiting for Author=我想要此功能,但是您将其编码错误,在您按照我建议的方式修复它之前,我不会批准。并且Rejected=我不希望您要求我对此功能进行审查。
goku_da_master
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.