回购作者的Git pull请求礼仪 - 破碎的风格，无关的变化，部分修复

Question

我在GitHub上获得了第一个拉取请求。 https://github.com/Ark-kun/Piranha/pull/1

变化非常小且易于理解。 不幸的是，我有点过于完美。我看到这个拉取请求有3个问题：

1. 破碎的样式（标签代替空格;缺少花括号）
2. 拉取请求有两个不完全相关的修复 - 修复了NullReferenceException并修复了Mac OS X的库路径。
3. 虽然此请求修复了Mac OS X的路径，但它并没有为Linux修复它。

我该怎么办？

1. 请PR公主解决问题。
2. 只需自行移植并修复代码，然后关闭PR而不接受它。
3. 签出PR分支，修复样式和代码，合并固定分支。 （可能压缩提交）
4. 别的。

Answer 1

（1）和（3）都是合理的选择（如果你选择＃3，我不会压缩另一位作者的提交。）

我避免（2）因为它省略了归因 - 您可能希望鼓励人们对您的项目进行改进，并且归因在这方面很重要。

Answer 2

我认为第一个选项比其他选项更好，因为将存储库公开并有拉取请求应该可以帮助您完成项目，因此向那些试图提供帮助的人表示赞赏是件好事。仅通过移植代码和拒绝PR来隐藏他们的贡献不会鼓励他们做更多事情。所以我绝对避免选择2。 其次，自己修复代码会浪费你的时间。我知道，它只是一些键盘快捷键来修复缩进，但如果你让人们贡献代码，这与你的标准不符（这对于公共项目来说非常重要，而且一般来说也是如此） ，当然），你将来会有很多修复。

Github有很多选择将编程社会结合在一起，并且很容易将评论发布到PR或甚至代码行。因此，当你与你的贡献者互动时，你会让他成为一个更好的程序员，而他的下一个PR也会更好（希望:)），所以这符合你的利益。有时在这些评论中开始真正有趣的讨论，所以它可能是你们两个人的学习经历。

当然，如果修复非常小，你可以选择第三个＆＃34;动作＆＃34;并自己修改，但无论如何我都会联系你的贡献者。