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问题进行代码审查_Git_Github - Fatal编程技术网

使用github问题进行代码审查

使用github问题进行代码审查,git,github,Git,Github,在我的组织中,我们将源代码存储库从svn迁移到企业github。我们希望使用一个工具来管理代码审查,但我们正考虑使用Github问题来管理代码审查,而不是在我们的CI-CD堆栈中再添加一个工具 我们的存储库包含一个开发分支,所有开发人员在其中提交和推送他们的代码。构建和部署也通过开发分支进行 许多人给我们的一个建议是使用一个登台分支,在这个分支中开发人员签入他们的代码,对该分支进行代码审查,然后向开发分支执行pull请求。我们不喜欢这种方法,因为它将不必要地创建额外的pull请求步骤。如果我们允

在我的组织中,我们将源代码存储库从svn迁移到企业github。我们希望使用一个工具来管理代码审查,但我们正考虑使用Github问题来管理代码审查,而不是在我们的CI-CD堆栈中再添加一个工具

我们的存储库包含一个开发分支,所有开发人员在其中提交和推送他们的代码。构建和部署也通过开发分支进行

许多人给我们的一个建议是使用一个登台分支,在这个分支中开发人员签入他们的代码,对该分支进行代码审查,然后向开发分支执行pull请求。我们不喜欢这种方法,因为它将不必要地创建额外的pull请求步骤。如果我们允许有限的开发人员执行pull请求,就会在他们身上产生额外的瓶颈。如果每个人都有权执行拉请求,开发人员将绕过审查过程,直接完成拉请求

尽管如此,我们中的一些人正在考虑将“Github问题”用于所有代码审查任务。在开发分支的每一次推送中,开发人员都可以创建一个带有label=CodeReview的问题。这是个好主意吗


或者我们应该选择Crucible/Fisheye等解决方案吗?您可以在开发过程中的任何时候打开一个Pull请求:当您几乎没有代码,但想分享一些屏幕截图或一般想法时,当您陷入困境,需要帮助或建议时,或者当您准备好让他人查看您的工作时。通过在拉取请求消息中使用GitHub的@提及系统,您可以请求特定人员或团队的反馈,无论他们在大厅内还是十个时区之外

拉取请求对于促进开源项目和管理对共享存储库的更改非常有用。如果您使用的是叉和拉模型,拉请求提供了一种通知项目维护人员关于您希望他们考虑的更改的方法。如果您使用的是共享存储库模型,那么在将请求合并到主分支之前,请求请求可以帮助您开始代码审查和有关建议更改的对话

讨论并检查您的代码

打开拉取请求后,审阅您的更改的人员或团队可能会有问题或意见。也许编码风格与项目指南不匹配,更改缺少单元测试,或者一切看起来都很好,道具都井然有序。Pull请求旨在鼓励和捕获这种类型的对话

您还可以根据有关您的提交的讨论和反馈,继续推动您的分支机构。如果有人评论说你忘了做某事,或者代码中有错误,你可以在你的分支中修复它,并推动更改。GitHub将在统一请求视图中显示您的新提交以及您可能收到的任何其他反馈

合并和部署

一旦您的拉请求被审查并且分支通过了测试,就可以将代码合并到主分支进行部署了。如果要在GitHub上的存储库中进行合并之前进行测试,可以先在本地执行合并。如果您没有对存储库的推送访问权限,这也很方便

合并后,拉请求将保留代码历史更改的记录。因为它们是可搜索的,所以任何人都可以回到过去了解决策的原因和方式

注:
通过将某些关键字合并到拉取请求的文本中,可以将问题与代码关联起来。合并拉取请求时,相关问题也将关闭。例如,输入短语Closes#32将关闭存储库中的第32期。

您可以在开发过程中的任何时候打开拉取请求:当您几乎没有代码,但想分享一些屏幕截图或一般想法时,当您陷入困境并需要帮助或建议时,或者当你准备好让别人检查你的工作时。通过在拉取请求消息中使用GitHub的@提及系统,您可以请求特定人员或团队的反馈,无论他们在大厅内还是十个时区之外

拉取请求对于促进开源项目和管理对共享存储库的更改非常有用。如果您使用的是叉和拉模型,拉请求提供了一种通知项目维护人员关于您希望他们考虑的更改的方法。如果您使用的是共享存储库模型,那么在将请求合并到主分支之前,请求请求可以帮助您开始代码审查和有关建议更改的对话

讨论并检查您的代码

打开拉取请求后,审阅您的更改的人员或团队可能会有问题或意见。也许编码风格与项目指南不匹配,更改缺少单元测试,或者一切看起来都很好,道具都井然有序。Pull请求旨在鼓励和捕获这种类型的对话

您还可以根据有关您的提交的讨论和反馈,继续推动您的分支机构。如果有人评论说你忘了做某事,或者代码中有错误,你可以在你的分支中修复它,并推动更改。GitHub将在统一请求视图中显示您的新提交以及您可能收到的任何其他反馈

合并和部署

一旦您的拉请求被审查并且分支通过了测试,就可以将代码合并到主分支进行部署了。如果要在GitHub上的存储库中进行合并之前进行测试,可以先在本地执行合并。如果您没有对存储库的推送访问权限,这也很方便

合并后,拉请求将保留代码历史更改的记录。因为它们是可搜索的,所以任何人都可以回到过去了解决策的原因和方式

注: 通过将某些关键字合并到拉取请求的文本中,可以将问题与代码关联起来。当你拉