|
...
|
...
|
@@ -2,7 +2,6 @@ |
|
|
|
|
|
|
|
## 亲们，请统一编程风格！ (PHP)
|
|
|
|
=======================
|
|
|
|
> <?php
|
|
|
|
> class Yoho
|
|
|
|
> {
|
|
|
|
> public static function callApi($param)
|
|
...
|
...
|
@@ -13,7 +12,6 @@ |
|
|
|
> }
|
|
|
|
> }
|
|
|
|
> }
|
|
|
|
> ?>
|
|
|
|
|
|
|
|
* 推荐在编辑器中针对项目设置好 *
|
|
|
|
* 注意花括号的位置 *
|
|
...
|
...
|
@@ -31,14 +29,14 @@ |
|
|
|
|
|
|
|
* 代码推送到远程主干之前，必须经过其他同事帮你审查! 只有审查通过后才允许推送到远程. *
|
|
|
|
|
|
|
|
场景1: 小A 完成一个功能, 需要合并到远程主干，找小B 帮他审查代码，小B发现 小A 代码里有好多缺陷，
|
|
|
|
需要提出来，让小A修改. (小A不修改，不允许帮他合并到主干)
|
|
|
|
** 场景1: 小A 完成一个功能, 需要合并到远程主干，找小B 帮他审查代码，小B发现 小A 代码里有好多缺陷，
|
|
|
|
需要提出来，让小A修改. (小A不修改，不允许帮他合并到主干) **
|
|
|
|
|
|
|
|
场景2: 小C 完成一个功能, 需要合并到远程主干，找小D 帮他审查代码，小D 发现 小C 代码写得很不错，都懂了，
|
|
|
|
这时小D 可以把代码合并到主干，小D 需要 写上code review by 小D: 代码相关的描述内容 (合并到远程之后，小C 和 小D 需要对这个功能负责! )
|
|
|
|
** 场景2: 小C 完成一个功能, 需要合并到远程主干，找小D 帮他审查代码，小D 发现 小C 代码写得很不错，都懂了，
|
|
|
|
这时小D 可以把代码合并到主干，小D 需要 写上code review by 小D: 代码相关的描述内容 (合并到远程之后，小C 和 小D 需要对这个功能负责! ) **
|
|
|
|
|
|
|
|
场景3: 小C 代码已合并到主干，突然间发现还有一个需要改进的地方需要调整，则检出主干代码, 再修改，
|
|
|
|
修改好之后请找同事帮助审查! 不要直接推远程，只有审查通过才可以推送到主干!
|
|
|
|
** 场景3: 小C 代码已合并到主干，突然间发现还有一个需要改进的地方需要调整，则检出主干代码, 再修改，
|
|
|
|
修改好之后请找同事帮助审查! 不要直接推远程，只有审查通过才可以推送到主干! **
|
|
|
|
|
|
|
|
## 重要特点
|
|
|
|
* 该文件 自 2016年1月1日起，WEB组 开始实施！*
|
...
|
...
|
|
