Review Board的使用v0.2
简介
Review Board是一个Code Review的辅助工具,同时使用tao-review board eclipse插件,可以简化整个code review的过程;Review Board的原理是将svn diff的结果以一种可视化的方式集中在网站中展示,提升code review的沟通反馈过程
一图胜千言
工作流程
在eclipse中开发测试完成提交svn前,使用集成在ecipse中的tao-review board插件,先提交到review board,找该项目对应的reviewer对提交svn前的代码进行评审,提出改进的意见反馈,接受反馈后继续重构代码,再次完成后再提交review board,如此往复,直到代码审核通过后再提交svn
其它
如果不使用eclipse可以手工生成diff文件,进行上传,eclipse只是集成了这个功能,方便使用。
另外review board也提供了命令行下的工具,具体参考
http://www.reviewboard.org/docs/manual/dev/users/tools/post-review/
具体操作步骤
- 注册自己的账号,建议用户名为姓名全拼
- 安装Tao-review board eclipse插件
- Subclipse的版本建议使用1.8.5
以上需要下载的资料在179 softlib下有下载
在eclipse下的设置
选择你正在开发的project右键选中鼠标所指该项,Review board 支持pre-commit & post-commit 两种模式的code review,我们使用前一种。
Reference:
http://en.wikipedia.org/wiki/Code_review
关于发送邮件参数的设置
测试服务器是否能发邮件的脚本
#!/usr/bin/env python
#encoding=utf-8
from django.core.mail import send_mail
from django.conf import settings
settings.configure()
settings.EMAIL_HOST="mail.360buy.com"
settings.EMAIL_HOST_USER="maolingzhi@360buy.com"
settings.EMAIL_HOST_PASSWORD="xxxx"
settings.EMAIL_USE_TLS=False
print "="*10
print settings.EMAIL_HOST
print settings.EMAIL_HOST_USER
print settings.EMAIL_PORT
print settings.EMAIL_USE_TLS
print settings.EMAIL_HOST_PASSWORD
print settings.EMAIL_SUBJECT_PREFIX
print "="*10
send_mail('Subject here', 'Here is the message.', 'maolingzhi@360buy.com',['maolingzhi@360buy.com'], fail_silently=False)
根据以上脚本来正确执行否来调整是否要勾选TLS等参数
开启日志输出
便于定位错误,需要设置和apache2启动的用户(我这是daemon)相一致的读写权限
2012-02-29