C#代码整洁之道:代码重构与性能提升
上QQ阅读APP看书,第一时间看更新

2.1 代码评审流程

通常,在代码评审过程之前要保证代码编译正常并能够满足相应需求,还应当确保代码通过所有的单元测试与端到端测试。一旦确信代码可以成功编译、测试与执行,那么就可以将其签入当前的工作分支中。在签入完毕之后就可以创建pull request[1]了。

此后,同行评审员将评审这些代码并提供评论和反馈。如果代码通过了评审,则代码评审结束,并可以将工作分支合并到主干中。否则如果同行评审失败,则提交者需要重新审视代码,并处理评审意见中的问题。

图2-1展示了同行评审的过程。

图2-1


[1]拉取请求,由于该英文术语使用广泛,因此后文中均写为pull request。——译者注