如何在github中讨论pull请求

Question

当我的同事或我不同意一段代码或方式在拉动请求中做事时，我有一点问题。我想知道如果审稿人不同意编码器，你是如何解决讨论的呢？

1）PR将不被批准且不会合并

或

2）编码员不接受评论并拒绝评论。无论如何，代码将被合并。

上下文是审阅者，拉取请求所有者与同一平面 开发人员都是该项目的所有者。

感谢您的意见

Answer 1

假设有两个拥有相同所有者特权的人，一个是PR的作者，另一个是评论者：

1. 测试

   首先，如果代码实际上解决了所需的问题，则应对其进行测试。如果没有，则无需合并。

2. 请求更改

   如果代码修复了问题或实现了新功能，那么设计问题就会出现，例如缩进，制表符/空格，愚蠢命名的变量，甚至是原始存储库可能试图保持接近的代码模式/样式指南。

3. 代码很好

   如果双方的代码都很好（算法+设计）并且还有人不赞成，那么这些问题就有很多：

   • 代码只是感觉不对
   • 不必要的功能
   • 内部问题 - 也许PR的评论者和作者彼此不喜欢

如果第三个“阶段”到来，那么在这种情况下，大多数人（具有相同级别的特权）将需要判断代码。

有点乌托邦式的想法，第三个人总能解决这种情况，因此这里有一些你应该习惯的结局：

• PR将保持开放并在将来解决

• PR将被关闭

  欢迎你做一个分支并研究这个功能，毕竟它是git的本质（参见DVCS）并且可能在未来的新PR中提出代码

• PR将合并