程式設計 /z/programming
6
請教 Code review 工具 (/z/programming)

目前公司進行 code review 的方式就是開 svn log 然後逐個 revision 掃過，有看見問題就用 google doc 註記下問題，雖然也進行的下去，但就是有點土炮而且感覺沒什麼效率。

因為公司嚴格要求不能把 code 上傳雲端，所以 Github or Bitbucket 之類的方案就被否決了。

我稍微Google了一下發現有 Jetbrain 的 Upsource1 ，或 Phabricator2 這類的產品，內含 code review 功能，只是沒用過也不知道哪個比較好用。

不知道大家公司的 code review 是怎麼進行的，有沒有什麼工具推薦?

1
https://www.jetbrains.com/upsource/
2
http://phabricator.org/
6 則討論 chchwy 張貼於

afutseng 積分 8 編輯於

推薦用 Phabricator，我們公司選用它而非 GitLab 的原因主要有這兩點：

  • 公司主流言語言是 PHP，如果要客製化（文末的例子）也比較容易

  • DevOps 同事不傾向採用 Ruby 方案

如果公司沒包袱， Phabricator 也能勝任 issue tracking system 的角色

Phabricator 做 Code Review 的優勢除了 line by line comment 之外，還可以對 commit 做 Accept / Raise Concern

我們的流程是：

  1. Reviewer 對某段 code 有疑問，Raise Concern

  2. Author 回覆，決定處理方式：直接修正或是跟 Reviewer 辯論

  3. Reviewer 認同處理方案，按 Accepted 結案；不認同的話繼續回覆，甚至拉其他人討論

最近參考 Phabricator Bot 文件1寫了一支 Bot daemon，Reviewer 回覆 commit 就會噴訊息到 HipChat 並 mention author，讓比較習慣使用 IM 工具而非 email 的人可以及早收到 Reviewer 的意見 (Phabricator 可以設定有人回覆就寄信出來）

1
https://secure.phabricator.com/book/phabdev/article/chatbot/
回應

kentchiu 積分 0

github可以pull request 進行review reivew完沒問題再進行 merge

回應

chchwy 積分 0

我也很想用 github 阿，可是公司不允許XDDD

回應

negaihoshi 積分 0

你可以自己 host gitlab

回應

ymhuang0808 積分 0

覺得 GitLab 安裝蠻方便的

用 Google Doc 是也可以，感覺 tracking issues 不是很方便

回應