备注:认为code review没有用的人,只能说明他们没有用过code review,适当的code review能减少代码隐藏的漏洞,使代码更规范,更能快速提高自己的技术水平。之前在网上找了很多代码code review的工具,也试过phabricator 这样的工具,感觉还是GitHub自带的pull request比较好用。
个人认为任何事情都不用太依赖于一种方式(不要太处女情结),否则效率会受很大影响。例如:code review的目的是控制代码质量,减少不必要的隐藏漏洞,互相学习编码习惯和技巧。所以个人觉得以下场景就可以大胆的略过code review了:
1、更新第三方库
2、修改版本号
3、添加图片资源等其他操作
Pull Request 流程总结:
总流程文字介绍:
多人合作开发使用Pull Request的方式是:项目的每一个版本开发需要建对应的一个版本号例如:v1.0.0 ,然后每个人需要基于这个分支建自己的小分支例如:v1.0.0_name1 ,这样的话你就可以在v1.0.0_name1分支上开发了,当想把自己分支的代码提交到当前项目的分支v1.0.0 中的时候,就可以用 Pull Request 来提交,然后code review的管理者就可以看到有 pull request请求,然后管理者可以查看提交的内容,可以评论,和通过。当管理者通过后,你的提交就自动进入了v1.0.0分支里面了。
总流程截图步骤:
一、用户在当前项目下点击 New pull Request按钮:
注意:基于的分支和自己的临时分支一定要选择正确


二:管理者会收到通知,并且可以在Pull request里面查看,如图:
管理者可以在Files changed里面查看更新内容,在Conversation里面评论或者点击 : Merge pull request按钮接受提交,此时你的代码就自动更新到 v1.0.0分支了。





我是用一个账号演示的,多个账号一样道理。如果你没有该项目的创建分支的权限(不是自己的项目)可以先Fork到自己的账号里面,同样可以pull request。
网友评论