ReviewBoard进行Code Review

一、标题

  • Review Board 简介
  • Code Review流程
  • 手动创建Review请求
  • Tao-ReviewBoard插件
  • Code Review评审流程
  • Code Review权限控制

二、Review Board简介

  • 代码审查(Code Review)不但可以提高质量,而且还是一个知识共享和指导的极好的手段。不幸的是,准备工作的辛苦和工具支持的缺乏让代码审查很容易被延至“稍后再议”。Review Board的目标便是改变这一现状,它所提供的应用程序可以支持代码审查流程。一些应用程序的特性包括:
  1. 详细的审查请求将所有代码审查所需要的信息都集合起来,包括代码的修改和屏幕截图
  2. 一个支持将评论直接写至对应的代码行的diff查看器
  3. 上下文讨论
  4. 一个汇总审查和被审查需求的面板
  • Review Board在审查补丁(Patch)方面表现完美。一个叫做“提交审查”的工具被用来和SCM系统进行连接(当前支持SVN、CVS、Perforce、Git和Mercurial等),可以允许你请求一个将被提交的修改的审查。

三、Code Review 操作流程

四、手动创建Review请求

1、命令行创建Diff(也可以用tortoisesvn create patch)

2、登录reviewboard创建review request

五、手动创建Review请求(cont.)

1、填写必要信息后publish review request

六、Tao-ReviewBoard插件

1、Tao-ReviewBoard是在eclipse上开发的一款ReviewBoard代码评审插件,它很好的解决了安装和操作等问题,使reviewboard的使用更加方便。该插件目前支持ReviewBoard1.6及以上版本。现在,你已经不再需要在本地安装任何其它额外软件(甚至无需安装Subversion),也不再需要记住任何脚本和命令;在Eclipse中安装ReviewBoard插件后,提交和更新Request的操作将非常便捷,你只需要选中需要提交的代码或目录,然后右键Team进行相关操作即可完成Request的提交和更新(且支持Pre-Commit和Post-Commit方式)。Tao-ReviewBoard插件后续还将提供展示diff等诸多功能。

2、Tao-ReviewBoard用户手册下载地址:猛击这里

3、在线安装:安装地址: 猛击这里 (安装详细步骤可查看文档,安装前先必须安装subclipse)

七、Tao-ReviewBoard插件(cont.)安装步骤

 1、项目->右键->Pre-Commit Request to ReviewBoard

2、填写代码评审表单

①情况一:创建新的代码评审请求

②情况二:更新已有代码评审请求

3、提交成功

备注:点击链接进入ReviewBoard

八、Code Review评审流程

1、代码评审”提交成功后,相关人员收到提醒邮件

代码评审请求创建完成后,“请求提交人”“评审人(>=1)”“评审组成员(>=1)”都会收到提醒邮件。

备注:代码评审请求,从“创建”、“评审”到“关闭”的整个过程,相关人员都收到通知邮件

2、Code reviewer进入diff链接,进行code review, 如果对某行有意见,鼠标对准某一行点击即可出现comment框后填写意见:

3、此时code request发布者会收到comment的邮件,根据comment内容进行代码修改后重新update diff后点击FIxed;若不同意reviewer的意见则可以填写comment后,如果reviewer接受这种解释,则可以 点击Drop

4、如果code reviewer 对代码没有任何意见了,则可以ship it表示该review已经通过评审:

时间: 2024-08-04 05:11:55

ReviewBoard进行Code Review的相关文章

使用RBTool自动提交code review请求

使用RBTool自动提交code review请求 前言 让我们回想一下手工提交review请求的过程: 首先得用 svn diff > filename.diff 生成diff文件. 然后输入review board的网址,可能是 rb.companyname.com 然后需要输入你的账号密码进行登录验证. 然后你需要填写你的svn repository 地址,然后上传diff文件. 然后你进到review请求的详细页面,填写summary, description, test-done, g

后端code review finished 12-28

今天只有天赋和士杰有相应的后端代码的code review工作,因为并没有召开daily scrum.只是天赋和士杰对后端所有的代码进行了review,对代码进行了整理并删除了一些残留的0 refrence的代码.

code review的目的

Code review 是系统的检查程序源码,目的是在初始开发阶段找到和修正错误,提高软件质量和开发人员的技术水平. Java源码的Code review包括哪些那: 1.编程规范 2.面向对象设计检查 3.性能检查 4.资源管理:内存泄露 5.线程安全:多线程,死锁 6.处理流程:条件语句,循环结构 7.异常处理 8.数据库 有许多帮忙我们检查代码的自动化工具:比如PMD工具,http://pmd.sourceforge.net/pmd-5.1.1/ PMD可以帮我做的: PMD scans

我是如何进行code review的

众所周知,代码审查是软件开发过程中十分重要的环节,楼主结合自己的实际工作经验,和大家分享一下在实际工作中代码审查是如何开展的, 笔者水平有限,若有错误和纰漏,还请大家指正. 代码审查的阻力 我想不通公司不同部门对代码审查这项工作的重视程度还是不一样的,对于代码审查的阻力总结了以下几点: 国内的整体环境,国内的公司,尤其是互联网公司,讲究速度致上,软件开发的迭代周期周期短,速度快,因为竞争太大,开发的产品要求快速上线,对代码审查不是很重视,先上线,出了问题再解决. 公司的规模,大公司重视流程,把代

如何搭建gerrit开源code review工具

搭建环境:Ubuntu 14.04 一.环境准备 1.Java环境 gerrit依赖,用于安装gerrit环境. 下载:jdk-7u79-linux-x64.tar.gz http://www.oracle.com/technetwork/java/javase/downloads/jdk7-downloads-1880260.html 安装:sudo tar zxvf ./jdk-7u79-linux-x64.tar.gz -C /opt 配置:vim ~/.bashrc export JAV

如何在python脚本开发做code review

在软件项目开发中,我们经常提到一个词"code review".code review中文翻译过来就是代码评审或复查,简而言之就是编码完成后由其他人通过阅读代码来检查代码的质量(可编译.可运行.可读.可维护.可复用),这些性质都比较抽象,但是一般都可以通过以下的检查点来实现: 检查代码的命名方式是否符合规范,代码的可读和可维护必须要求所有参与编码的同事使用的命名有统一的规范(注意每个人有自己的代码风格,但是要符合可读性的代码规范): 检查代码的注释,注释一般包括:1.类要有类用途和使用

iOS从零开始 Code Review

http://www.cocoachina.com/ios/20151117/14208.html 这篇帖子不是通篇介绍Code Review的方法论, 而是前大段记录了我们团队怎么从没有这个习惯到每天都进行review的过程, 后小段给出了我的一些建议. 希望能对诸位的团队有所帮助. 最初来到这个新组建的团队是木有code review的. 头说, 这个月你来搞吧. 当我第一次知道必须得搞review的时候, 其实我是拒绝的! 因为我觉得…呀…你不能叫我马上搞立马搞, 第一, 我要试一下, 我

Friend's Code Review

代码: ``` package lservice; import java.util.Date; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; import org.apache.struts.action.Action; import org.apache.struts.action.ActionForm; import org.apache.struts

Code Review for SSIS package

以下是我对SSIS包进行code review的一些建议,如果有其他更好的方案欢迎拍砖. A. 查看是否使用了最优的解决方案 1. 最优的结构视图 2. 解决方案,包,任务,组建,参数的命名使用了易读的命名方式 3. 遵循了最优的设计,优化,调整方案 B. 配置 查看是否所有的配置已经成功,并且能够从外部和父包中获得正确的配置信息. C. 查看能否通过以下测试 1. 正常的场景 查看到所有的表数据/文件已经生成并且是正确的 查看所有的数据在表中没有被截断或有不需要的空格/字符 重新执行包,看是否