在一个Github项目中,我看到一个人在Pull Requests
部分提出了拉取请求,但我发现他犯了很多错误,所以他的代码需要纠正。而不是让他改变他的代码,是否有可能自己做(也就是说,我想自己纠正他的代码)?怎么样?
我也是该项目的成员,我是新人:)
答案 0 :(得分:3)
假设原作者的反应足够,那么最好让他们去修复它(除非改变是一个明显的拼写错误修复,也许)。他们可能有理由以他们的方式写作。
如果作者没有回应,而您决定是将其修复到位还是使用新提交,请问问自己历史记录的更清楚的方式。如果提交会引入回归,您应该将其修复到位(以避免破坏可分割性)。如果它处理某些情况并且您只想添加更多案例而不会显着更改现有代码,那么添加新提交可能会更好。
如果您以任何方式修改现有提交,请务必在提交消息中留言,说明您更改的内容。一种典型的风格是在方括号中留下一个注释,与任何Signed-off-by:行分组(如果适用): [your.email@example.com:为了清晰起见,将新案例重构为一个函数] 例如:linux.git commit b44129b3。
通过https://www.quora.com/GitHub-Is-it-bad-etiquette-to-change-someones-pull-request-before-committing
答案 1 :(得分:1)
在最初从派生创建拉取请求时或创建拉取请求之后,拉取请求创建者可以对每个拉取请求赋予这些权限。
有关github帮助页面的更多详细信息:Allowing changes to a pull request branch created from a fork
PR的作者可以修改现有的拉取请求,以使存储库维护者对您的分支进行提交。