天天看點

Review Board的使用v0.2

Review Board的使用v0.2

Review Board的使用v0.2

簡介

Review Board是一個Code Review的輔助工具,同時使用tao-review board eclipse插件,可以簡化整個code review的過程;Review Board的原理是将svn diff的結果以一種可視化的方式集中在網站中展示,提升code review的溝通回報過程

Review Board的使用v0.2

一圖勝千言

工作流程

在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/

具體操作步驟

  1. 注冊自己的賬号,建議使用者名為姓名全拼

    http://10.10.243.104:8080/reviewboard/account/register/

  2. 安裝Tao-review board eclipse插件

    http://code.taobao.org/p/tao-reviewboard/wiki/index/

  3. Subclipse的版本建議使用1.8.5

    以上需要下載下傳的資料在179 softlib下有下載下傳

    在eclipse下的設定

    Review Board的使用v0.2
    選擇你正在開發的project右鍵選中滑鼠所指該項,Review board 支援pre-commit & post-commit 兩種模式的code review,我們使用前一種。
    Review Board的使用v0.2
    Review Board的使用v0.2

    Reference:

    http://en.wikipedia.org/wiki/Code_review

    關于發送郵件參數的設定

    Review Board的使用v0.2

    測試伺服器是否能發郵件的腳本

    #!/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="[email protected]"

    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.', 'maoling[email protected]',['[email protected]'], fail_silently=False)

    根據以上腳本來正确執行否來調整是否要勾選TLS等參數

    開啟日志輸出

    便于定位錯誤,需要設定和apache2啟動的使用者(我這是daemon)相一緻的讀寫權限

    Review Board的使用v0.2
    2012-02-29

posted on 2012-02-29 15:08  lexus 閱讀( ...) 評論( ...) 編輯 收藏

轉載于:https://www.cnblogs.com/lexus/archive/2012/02/29/2373483.html

繼續閱讀