Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
xindoo
eng-practices-cn
提交
5dfa3630
E
eng-practices-cn
项目概览
xindoo
/
eng-practices-cn
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
E
eng-practices-cn
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
未验证
提交
5dfa3630
编写于
1月 19, 2020
作者:
X
xindoo
提交者:
GitHub
1月 19, 2020
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #9 from yuxiangtong/patch-1
Update index.md
上级
4356a50f
29d5299e
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
2 addition
and
2 deletion
+2
-2
review/index.md
review/index.md
+2
-2
未找到文件。
review/index.md
浏览文件 @
5dfa3630
...
...
@@ -15,7 +15,7 @@ Code Review是一些人提交一些代码片段,供另外一些人审阅的流
-
**设计**
: 代码是否设计良好并且适合你们的系统?
-
**功能性**
: 代码功能是否和开发者预期一致?这种方式是否对用户友好?
-
**复杂性**
: 代码能不能更简单? 其他开发者能否快速理解并在未来很容易地使用这段代码?
-
**测试**
: 这段代码是否有
真
确和设计良好的自动化测试样例?
-
**测试**
: 这段代码是否有
正
确和设计良好的自动化测试样例?
-
**命名**
: 开发者有没有正确地对变量、类、方法等命名?
-
**注释**
: 注释是否清晰有用?
-
**代码风格**
: 代码风格是否遵循
[
谷歌代码风格指南
](
http://google.github.io/styleguide/
)
?
...
...
@@ -34,4 +34,4 @@ Code Review是一些人提交一些代码片段,供另外一些人审阅的流
## 参见
-
**[评审者指南](reviewer/)**
: 代码评审者的详细指南。
-
**[开发者指南](developer/)**
: 提交变更评审的开发者的详细指南。
\ No newline at end of file
-
**[开发者指南](developer/)**
: 提交变更评审的开发者的详细指南。
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录