net源码关于 Pull Request 的十个

重庆中考论坛zslpsh,重庆学校zslpsh,重庆学校zslpsh,重庆市中学生网,重庆中学生网,重庆中学生网站,重庆中学生网好,重庆中学生学习网,重庆中学学习网,重庆中学生网家教,重庆中学生论坛,重庆学生zslpsh,重庆高考论坛zslpsh,重庆中学排名zslpsh,重庆中学zslpsh,重庆中学生,重庆学生网中考,重庆中考zslpsh,重庆中学生交友,重庆初中生,重庆中学生学习网,重庆土话网,西南云南方言网,最新电影淘娱淘乐,tianyanmao.cn,重庆18680好,岳阳yy房产网,重庆18680,云南西南方言网,网店taoyutaole,电影淘娱淘乐,最新电影淘娱淘乐,电影淘娱淘乐,娱乐资讯taoyutaole,影视淘娱淘乐,taoyutaole笑话,taoyutaole淘娱淘乐,娱乐taoyutaole,时尚taoyutaole,重庆生活新闻,贵州西南方言网,027旅游新闻网,重庆特产18680,0871昆明旅游人才网,重庆生活18680好,观赏虾之家zadull,022天津交友,022天津交友网,水草造景zadull,18680重庆特产,0755深圳交友网,0755深圳旅游招聘,0755深圳旅游招聘网,0755深圳旅游新闻

　　阿里百川梦想创业大赛，500万创投寻找最赞的APP

　　Pull Request 是Bitbucket、GitHub等源代码托管系统为了方便开发者之间协作而提供的一个功能，它提供了一个用户友好的 Web 界面来帮助审查人员进行代码审查。开发人员可以通过 GitHub 发出 Pull Requests 要求请求他人将程序拉下来进行代码审查。一个好的 Pull Request 不仅仅只是代码的事情，还牵涉到代码审查者对代码的审查，所以开发者不仅要写出好的代码，还必须迎合审查者的审查工作，才能给使得自己贡献的代码顺利通过审查并合并到 master 分支。现对丹麦的程序员、软件架构师、顾问Mark Seemann在自己博客中发布的一篇题为《关于 Pull Request 的十个》的文章进行一个全面的整理，以供读者阅读和参考。具体内容如下：

　　一个小且集中的 Pull Request 会使得提交的代码更加容易通过审核。据 Mark Seemann 根据自己的经验透漏，对提交代码进行审查所花费的时间是随着代码大小呈指数增长，而非线性增长；Pull Request 多大才合适与 Pull Request 做了什么相关，最好少于 12 个文件的改变。如果 Pull Request 非常大，审查者就需要安排连续、相对比较长的时间进行审查，就会造成审查过程的延迟。

　　就如软件设计模式中的单一责任原则所指：一个类只负责一个功能领域中的相应职责，因此一个 Pull Request 也应只关注一件事情。如果一次 Pull Request 做了多件事情的话，将会增加审查过程的延迟、审查被的可能性。

　　代码审查者会使用不同的审查工具来审查提交的代码，并且 GitHub 和 Stash 还提供了不同形式的视图，这样就使得代码审查者能通过不同视图非常方便来审查用户的提交。如果代码行比较宽的话，审查者就不能在一个屏幕或者半个屏幕中来阅读代码，不得不拖动滚动条来阅读代码。为了使得代码比较容易审查，每行代码最好少于 80 个字符。

　　就算自己觉得应该改变当前代码的格式以适合自己的风格，但是请不要这么做。在源代码上，用户对每个字节的改变将会在不同的审查视图显示出来。有些审查者会选择忽略空格改变，但是有些审查者会审查这些代码，对这些格式化引起的代码审查属于不必要的审查。如果真需要解决空格问题的话，就需要在其他文件里移动代码、改变格式或者对代码做其他样式改变，并在 Pull Request 注释中给出相应的说明。

　　在提交 Pull Request 时，应该首先在自己的电脑上进行编译构建。在编译构建过程中，务必注意编译过程出现的，要把当作错误来对待，这些可能不会编译，就有可能被忽略。然而，当用户 Pull Request 操作引起了很多编译的话，代码审查者就有可能对应的提交。

　　利用文档对代码进行注释、对自己的代码直接进行注释、利用版本控制器提供的提交信息功能备注信息、利用 Pull Request 管理系统（如 GItHub 或者 Stash）添加自定义的 Pull Request 注释信息。

　　不同审查者对 Pull Request 有可能具有不同的观点，这就会导致颠簸的情况，就需要用户移除冲突的提交和推送修改的分支，并备注有效的信息。

　　稿源：infoq

　　Git@OSC同样提供 Pull Request 功能，欢迎使用。