Github git pull请求——如果旧的PR不被接受,是否应该创建新的PR

Github git pull请求——如果旧的PR不被接受,是否应该创建新的PR,github,gitlab,Github,Gitlab,我对如何处理PR/MR流程工作流有些不清楚 比如说,我正在github或gitlab中创建一个功能分支,并向我的审阅者提交PR/MR 假设我的PR不被接受(审核人给出了我需要解决的意见),他/她是否应该关闭PR,然后,我是否必须为此功能分支提出另一个PR 或者 我应该让我的审阅者重新审阅功能分支的最新版本(因为它有新的提交)并将其合并吗?您应该添加新的提交以解决审阅者的问题。如果审阅者在上有通知,则在您每次进行新更改时,他们都会收到电子邮件。如果提交更改了他们评论的文件部分,那么他们的旧评论将无

我对如何处理PR/MR流程工作流有些不清楚

比如说,我正在github或gitlab中创建一个功能分支,并向我的审阅者提交PR/MR

假设我的PR不被接受(审核人给出了我需要解决的意见),他/她是否应该关闭PR,然后,我是否必须为此功能分支提出另一个PR

或者


我应该让我的审阅者重新审阅功能分支的最新版本(因为它有新的提交)并将其合并吗?

您应该添加新的提交以解决审阅者的问题。如果审阅者在上有通知,则在您每次进行新更改时,他们都会收到电子邮件。如果提交更改了他们评论的文件部分,那么他们的旧评论将无效(在Github中,它们将被隐藏在视图之外)。然后他们可以继续检查新代码,然后重复。没有必要创建新的公关

其他说明:

  • 当你做出新的改变时,不要放弃评论。解雇通常用于同事休假或无法审查新变化时

  • 临时关闭PR通常是为了清理PR列表,但如果您的PR处于可查看状态,则可以稍后重新打开

  • 一些团队希望您在更改完全不同的情况下创建新的PR(即,您当前的方法不起作用,您应该从头开始)


我认为您应该让PR保持开放状态,处理请求的更改,并推动您的分支机构。应更新PR,并且应针对新提交运行任何CI。然后审核人应再次检查PR(可能会从推送中收到通知…)。提出新请求将使您失去审核历史记录(如果2个PRs未以某种方式链接)