upsource使用说明之代码审查规则

一、代码审查的意义

代码审查简单的描述就是让其他同事来检查你的代码，其目的在于找到开发初期所忽略的错误，从而提高代码的整体质量，降低风险，减少代码隐藏的一些问题。同时通过同事间的合作可以相互学习并取得进步。代码提交者很有可能从该工作中得到反馈，并意识到可能存在的问题和需要改进的部分；而审查者也可以通过阅读他人代码学到新的东西，并找出适用于自己的工作方案。

二、使用Upsource进行代码审查

1. Review审查流程

由有权限的同事针对某个或者多个代码提交操作发起一次代码审查流程，然后审查人员添加跟这次审查流程相关的Reviewers和Watchers，这里的 Authors、Reviewers、Watchers 都是复数形式表示都有可能是多个人员。

一次Review也可以添加多个Revisions(修订，可以理解为一次版本提交)。

一般情况下将这次Review中关系关系很密切的人设置为Reviewer，然后这次Review需要知道的人设置为Watcher，假设某个Review是关于某个接口的参数修改，我们可以把接口提供方的调用方都设置为Reviewer，然后跟这个接口关系不够密切但是需要知晓的人设置为Watcher。各个参与人员的角色没有很明确的划分界限，可以根据实际情况来确定。
Review的参与人员都有权限回复别人提出的意见，Authors（代码作者）角色的人在这次Review是执行者，负责执行大家讨论的结果。

如果Authors 在IDEA中完成经过讨论的修改方案后，提交代码的时候可以设置Review的ID，表示本次提交是为了解决某个Reivew中提出的问题。

然后在upsource中就可以跟踪到某个相应review的解决结果了。

当 Authors 修改完成代码后， Reviewers 可以 Accept修改结果或者Raise concern ,当所有 Reviewers 都 Accept修改结果的时候，这个Review可以正常关闭，默认情况下 Authors也可以关闭Review，但是可以设置不允许 Authors 关闭。（系统管理员选择Edit Project -> Advanced -> 取消 Allow authors to close reviews的选项）

如果提交的时候忘记忘记勾选 reivew with upsource 选项了可以在提交完成后在Version Control ``Tab页面右击鼠标选择upsource 然后设置到对应的Review即可。