Warning: file_get_contents(/data/phpspider/zhask/data//catemap/2/github/3.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181
在修复github上“开源”回购的PR问题后,我是否应该“git rebase” 问题:_Github_Open Source - Fatal编程技术网

在修复github上“开源”回购的PR问题后,我是否应该“git rebase” 问题:

在修复github上“开源”回购的PR问题后,我是否应该“git rebase” 问题:,github,open-source,Github,Open Source,现在的情况是这样的,在我为github上的开源回购创建PR之后,审阅者提出了一些建议和讨论,在我修复这个讨论之后,我应该做一个重新基础还是仅仅做一个修复提交 比较: 缺点:审阅者很难验证我刚刚提交的更改,很容易看到是否有修复提交,审阅者需要再次完成代码,这对他来说可能不是个好主意 优点:git树将非常干净 结论 哪一个是最好的选择 考虑一个公共回购协议,有很多审查者,他们可能没有太多时间来处理它。你的缺点是正确的,更难看到你修复了什么。这使你的公关更容易审查 在本例中,进行fixcommit

现在的情况是这样的,在我为github上的
开源
回购创建PR之后,
审阅者
提出了一些
建议
讨论
,在我修复这个
讨论
之后,我应该做一个
重新基础
还是仅仅做一个
修复提交

比较:
  • 缺点:审阅者很难验证我刚刚提交的更改,很容易看到是否有
    修复提交
    ,审阅者需要再次完成代码,这对他来说可能不是个好主意
  • 优点:git树将非常干净
结论 哪一个是最好的选择


考虑一个公共回购协议,有很多审查者,他们可能没有太多时间来处理它。

你的缺点是正确的,更难看到你修复了什么。这使你的公关更容易审查

在本例中,进行
fix
commit。此流程包含Github上的其余对话。如果他们正在使用它,它将与这些很好地配合

至于pro,一个更简单的历史记录,fix提交以响应代码审查是可以的。在推送之前进行简单的修复和输入错误提交,您可以使用交互式重基来消除这种情况


如果项目有一个特定的偏好,那么就使用它。

如果master有更多的提交,在刚刚创建的
PR和解决PR问题后的
之间呢。在这种情况下,似乎最好使用rebase。你的意思是先推送一个
修复提交
,如果它通过审核,告诉审核人在合并之前我需要一个rebase,然后再做rebase吗?@JerryZhou我建议询问项目他们希望你做什么。