【问题标题】:Is it okay to update the Pull Request with commits before code is reviewed and commented on?在审查和评论代码之前,是否可以通过提交更新拉取请求?
【发布时间】:2018-09-03 11:45:37
【问题描述】:

我在工作完成后创建了我的分支的拉取请求,并将其分配给代码审查员。之后,我打开了拉取请求并观察了我的更改。在任何人都可以查看之前,我发现了 1 或 2 个简单的错误。我已经在本地更正了它们。如果我提交并推送它们,拉取请求将得到更新。

在代码审查员审查和评论之前更新拉取请求在专业上是否正确?

提前致谢。

【问题讨论】:

  • 这真的取决于你和谁一起工作,但我认为这应该不是问题。如果您必须更改代码,那么您还能做什么。最好先修复它,然后让您的审阅者查看损坏的代码。您应该在发送之前尝试捕捉“简单错误”,因为如果它一直发生,它看起来有点草率。

标签: git bitbucket git-branch git-commit pull-request


【解决方案1】:

如果您认为需要更新拉取请求,我认为您没有任何问题。我一直这样做,总的来说,我认为您和您的审稿人都会从中受益。您会从中受益,因为这意味着双方以后将花费更少的时间来处理这些错误。如果你现在修复错误,你只需要稍后再做。

【讨论】:

    【解决方案2】:

    是的,可以这样做。

    github 的人鼓励尽早(甚至在开发完成之前)创建拉取请求甚至是一个很好的做法,这样,讨论就可以尽早开始,并且可以共同做出开发选择。 ..

    【讨论】:

      猜你喜欢
      • 2017-10-28
      • 2016-06-27
      • 2018-01-19
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 2021-11-25
      • 2017-05-03
      • 1970-01-01
      相关资源
      最近更新 更多