Github - 合并被阻止(混乱/误导性错误信息)

18
我创建了一个拉取请求,我的审批人请求进行一些更改,我标记了这些更改并点击了“重新请求”按钮,而没有查看合并是被阻止还是未被阻止。所以现在为了解除合并阻塞,审批人需要再次审查并批准更改吗?因为下面的消息没有说需要由审查者批准。

一旦请求的更改得到解决,合并可以自动执行。

合并已被阻止

2个回答

19
是的,您需要让您的审阅者批准更改。仅因为您认为您已经解决了问题并不意味着审阅者同意,而GitHub 知道它们已被解决的唯一方法是获得审阅者的批准。他们可能不同意并请求更多更改。
这个功能的部分原因是提供有效控制,在许多受管制行业中,在代码可以合并或部署之前需要进行控制。如果作者可以为自己的更改背书,那么这将不会非常有效。

一旦我解决了所请求的更改,是否必须重新请求评审人再次进行审核,还是他会自动收到通知而无需重新请求? - Derrick
1
最好通过界面重新请求审核。通常,评审人员会收到您所做的更新通知,但至少我倾向于忽略这些通知,因为人们经常在准备重新审核之前推送大量更新。 - bk2204
2
当请求更改的开发人员去度假时会发生什么?这正是我目前正在处理的问题。 - Tim Hysniu
4
这就是为什么信息“一旦处理请求的更改,合并可自动执行。”是错误的原因。无法“自动”执行任何合并操作。不,你需要审阅者审核你的额外更改。这并非自动完成。来吧,GitHub,修复这个信息。 - Zoltan Fedor
2
审核人员“解决了对话”,但这还不够,GitHub让我们一筹莫展,无法合并,除非我知道需要做什么,否则可能会无端打扰同事。 - wranvaud
@ZoltanFedor 在这个上下文中,“自动合并”意味着您不需要解决合并冲突。这与所请求的更改无关。 如果没有待处理的更改请求,提示只会说“可以自动执行合并”。 这条消息的问题在于它没有说明谁需要处理这些更改。 - undefined

6

最好情况下,请求更改的审阅者应该重新审阅。
但是...如果原始审阅者不可用,则可以联系其他人审核您的更改并代替他们批准。


override requested-changes

***当然,这应该负责任地使用 :)


网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接