Gerrit #TechLunch

Transcript

1. Gerrit Takuya Shiotari

2. コードレビューとは ソフトウェア開発工程で見過ごされた誤りを検出・ 修正するためにソースコードの体系的な検査を行 うこと

3. 良いコードを書くための5つの習慣 ▪ 読む ▪ 書く ▪ 道具を磨く ▪ 知る ▪

   聞く コードレビュー

4. コードレビューの勧め 【コードレビューの必要性】 ▪ 正しく機能を作りこんだか を確認して、テスト中毒か ら脱却する ▪ 読みやすさ、保守性が高 いコードを書いているかを 確認して、デバッグ・保守・

   エンハンスのコストを抑え る

5. コードレビューのメリット • 早期バグ発見 • リファクタリング • レビュアー／レビュイーのスキルアップ • コーディング規約の強制 •

   新人育成 • システムの引き継ぎを容易にする

6. 効率良くコードレビューを出来ていますか？

7. 現状感じている問題点 • デプロイ後のコードをレビューでは後の祭り • レビュアーがレビュイーにどのバージョンのどの ファイルのどの行番号っていうのを伝えるのは 手間 • どのバージョンがレビュー済みかわかりにくい

8. Gerrit

9. Gerritとは • Google社員が開発している • Androidのオープンソースプロジェクトで利用さ れている • rietveld(オープンソース版 Mondrian)の後継？ •

   名前の由来はオランダの建築家ヘリット・トーマ ス・リートフェルト(Gerrit Thomas Rietveld)から 来ている

10. Gerritの特徴 • Webベースのコードレビューシステム • Gitを使ったプロジェクトのオンラインコードレビュー • 変更を並べて表示してレビューしやすくしている • レビュアーはインラインにコメントを追加することができる •

    レビュイーはコミットをあるブランチ(refs/for/master)に PUSHして、レビュアーがレビューして承認すると自動的に masterブランチにマージされる

11. DEMO

12. 考察 • レビューそのものを資産として残すことができる • レビューで承認されないとマスターにマージでき ないので、致命的なバグを未然に防ぐことがで きる • Jenkinsとの連携もいい感じ •

    gitの使い方に慣れてないと難しい • ドキュメントが少ない

13. 次週のテーマ ▪ AngularJS ▪ factor

14. Mondorian Pythonの開発者Guido Van Rossumが開発 Googleが採用しているSCMのPerfoce用

15. rietveld • オープンソース版Mondorian • 開発者はMondorianと同じくGuido Van Rossum • SCMにSubversion

16. Crucible • ConfluenceでおなじみAtlassian製 • http://www.atlassian. com/ja//software/crucible/overview

17. 宍道湖（しんじこ） • Ruby on Rails製 • Google App Engine上で動作 •

    rietveldのクローン • http://code.google.com/p/shinjiko/