这篇文章的内容由github官方提供,指导你如何在github上进行代码审查和如何让别人审查自己的代码。
针对所有人的审查
- 接受这样的事实:很多编程上的主张都是一种个人观点。应该讨论它们的利与弊,提出你的倾向观点,迅速的达成一种解决方案。
- 提问,而不是命令。(“把这个变量命名成
:user_id
你觉得怎样?”) - 请求说明。(“我不明白。你能解释一下吗?”)
- 避免代码的归属之争。(“我的”,“不是我的”,“你的”)
- 避免使用一些会被认为是有关人身特征的词语。(“笨蛋”,“愚蠢”)要把所有人都看作是有魅力的、聪明的、善意的。
- 要明确。要记着并不是每个人都能理解你的意图。
- 要谦虚。(“我不能确定——我们来分析一下。”)
- 不要用夸张修辞语。(“总是”,“从不”,“永远”,“毫无…”)
- 不要讽刺。
- 展现真实的你。如果你不是幽默型的人,不喜欢使用一些表情符号或动画gif图,不要勉强。如果你是这种人,请自信的发挥。
- 如果有太多的“我不理解”或“另一种方案:”的评论,请专门针对这个人进行交流。可以把你们线下的交流总结成一个帖子附在后面。
让别人审查你的代码
- 对审查者的建议表示感激。(“谢谢提醒。我会把它改正。”)
- 理解审查是对事不对人。审查的是你的代码,而不是你。
- 解释为什么代码写成这样。(“因为xxx原因我才写成这样。如果我把这个类/文件/方法/变量改个名会更清晰些吗?”)
- 整理所作的改动,在以后的迭代中重构它们。
- 在做修改的版本上注明代码审查的链接。(“Ready for review:http://github.com/organization/project/pull/1″)
- push提交要基于最早的一轮反馈,并形成一个独立的分支。等这个分支上的任务完全完成了再合并。这让审查者能够根据早先的反馈找到你的单独的更新。
- 努力站在审查者的立场上理解。
- 争取回复每个评论。
- 直到最后一个人退出登录后再合并分支。
- 直到持续集成测试(TDDium, TravisCI,等)告诉你这个分支的测试套件通过后再合并分支。
代码审查的过程
先要清楚你提交的代码的必要性(是修补bug,提升用户体验,重构…)。然后:
- 针对你感觉非常好的地方以及不是很好的地方与作者交流。
- 找出既能解决问题又能简化代码的方法。
- 如果讨论变得过于哲学或理论,把讨论转到线下,做成一个有规律的每周五下午的讨论会。同时,是否采用你提出的实现方案,让作者自己做决定。
- 提出你的实现方案,但要表现出作者也在考虑这种方案。(“你觉得这里用一个自定义校验如何?”)
- 努力理解作者的立场。
- pull请求登出时,加一个 :thumbsup: 或“可以合并了”的注释。
关于程序风格样式的评论注释
审查者应该对那些不符合样式指导的地方进行注释。例如这样注释:
[Style](../style):
> 按名称的字母顺序排列多个路由。
对上面这个提醒的一个回复的例子:
哦。你眼真尖,谢谢。已在 a4994ec 修复。
如果你不同意某个指导原则,请在指导repo里创建一个问题,而不要再代码审查中争论它。同时,请运用这个指导原则。
相关推荐
微信小程序demo:github博客(源代码+截图)微信小程序demo:github博客(源代码+截图)微信小程序demo:github博客(源代码+截图)微信小程序demo:github博客(源代码+截图)微信小程序demo:github博客(源代码+截图)微信...
Github yolov5源代码
github开源键盘控制代码
怎么看github上的旧代码.zip
我在GitHub上创建的本地仓库代码
gitHub是全球大型的开源社区之一,它提供了一个线上的代码托管服务,所以很多开源代码都会被托管到gitHub上。如果作为新时代的码农不知道这个就相当low了。 如果我们只是需要浏览或者下载源码的话,是不需要注册账户的...
Prettier GitHub 利用 `prettier`格式化GitHub评论中的代码
farcy:针对github拉取请求的代码审查机器人 定义: 一种腺体,主要影响马和mu子的皮肤和浅表淋巴管。 虽然马和mu子具有讽刺功能,但这类动物可能更喜欢不带它们,而且它们对那些观看此类动物的人来说是一种眼神。 ...
微软github提供的DXGI 示例代码
github 使用指南 官方中文
Android的Github客户端源代码,适合用来学习
git-for-windows 2.15.1 github 官方版下载 压缩包 git-for-windows 2.15.1 github 官方版下载 压缩包
Flowable 6.4.0官方github源代码,可以编译通过的代码,对于学习Flowable的底层机制是一个必不可少的资源
Chrome扩展程序使用glob过滤GitHub代码审查中的文件
GitHub官方可视化操作工具,拥有GitHub账号即可登录使用
访问github github优质资源、代码
如何在github上传代码(图文解说) 用图文并茂的方式展示了在github上建立项目上传代码的全过程 简单,实用 waylau.com
GitHub官方中文版,可以在手机上查看管理自己的项目代码,可以分享项目共同学习,随时随地打开,方便快捷。
Github官方桌面客户端20191009最新windows(64bit)版,官方网址:https://desktop.github.com/
搜索应用要访问Github GraphQL API,请编辑.env文件,并将生成的访问令牌分配给VUE_APP_GITHUB_GRAPHQL_AUTH_TOKEN。 要创建,请参阅。项目设置npm install编译和热重装以进行开发npm run serve编译并最小化生产npm ...