我正在尝试建议使用GitHub的工作流程。 我想到的是，GitHub存储库中将有两个分支，分别是master和dev。其他开发人员会将其分叉给当地人。 说人A，在其本地的feature-1分支中开发了一个新功能，将其推到GitHub repos的dev分支，并发出拉取请求。 然后，所有开发人员都将GitHub dev分支拉入他们的本地语言并测试代码。 如果一切正常，则将dev分支合并到master。
我一直在对此进行研究，而我所获得的只是通过发出请求请求，其他开发人员只要阅读代码和评论就可以做什么。没有任何实际测试。
有人可以澄清吗？ 我认为通过执行拉取请求，所有开发人员都会得到通知，然后拉出代码，对代码进行一些实际编辑（不仅是读取和注释），然后将其推入与之前创建的相同的拉取请求中。
答案 0 :(得分：0)
最终，“审核时该做什么”取决于您的特定工作流程/团队。
如果您希望开发人员在过程中测试代码，则他们可以拉下source分支并使用它来运行代码进行测试。
如果您希望开发人员在检查时编辑代码，那么我对如何管理代码没有任何建议。我认为这根本没有意义（谁在审查审稿人？），但是我在下面提供了我的推理。
通常，带有“拉取请求”的想法是其他开发人员将审核更改并向原始作者提出改进建议-而不是自己进行更改。再说一遍，这是特定于工作流程/团队的事情，但是我个人认为对于另一个开发人员来说，只编辑我的PR而无需先与我交谈是很不礼貌的。
假定原始代码的编写者对主题/业务要求/技术限制具有最新知识，因此建议（及其推理）应由他们进行评估。同样，这是工作流/团队特定的事情。