git pull请求 - 如果不接受旧的PR，是否应该创建新的PR

Question

我对如何处理PR / MR流程工作流程缺乏明确性。

让我们说，我正在github或gitlab中创建一个功能分支，并向我的审阅者提交PR / MR。

假设我的公关不被接受（评论者发表了我需要解决的评论），他/她是否应该关闭公关，然后，我是否必须为此功能分支筹集另一个公关？

要么，

我应该要求我的审稿人重新审查我的功能分支的最新版本（因为它有新的提交），然后合并它？

Answer 1

您应该添加新的提交来解决审阅者的问题。如果他们有通知，审阅者将在您每次进行新的更改时通过电子邮件发送。如果提交更改了他们评论的文件部分，那么他们的旧注释将被无效（它们将在Github中隐藏）。然后他们可以继续审查新代码，重复冲洗。没有必要创建新的PR。

其他说明：

• 进行新的更改时，请勿关闭评论。解雇通常用于同事休假或无法审查新变更时
• 临时关闭PR通常用于清理PR列表，但如果您的PR处于可审查状态，则可以稍后重新打开PR列表
• 如果变化完全不同（即，您当前的方法不起作用，您应该从头开始），有些团队更喜欢您创建新PR