Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
Gerrit #TechLunch
Search
Livesense Inc.
PRO
April 23, 2014
Technology
0
40
Gerrit #TechLunch
2012/06/27(水) @ Livesense TechLunch
発表者:塩足 拓也
Livesense Inc.
PRO
April 23, 2014
Tweet
Share
More Decks by Livesense Inc.
See All by Livesense Inc.
27新卒_Webエンジニア職採用_会社説明資料
livesense
PRO
0
350
株式会社リブセンス・転職会議 採用候補者様向け資料
livesense
PRO
0
21
株式会社リブセンス 会社説明資料(報道関係者様向け)
livesense
PRO
0
1.4k
データ基盤の負債解消のためのリプレイス
livesense
PRO
0
400
26新卒_総合職採用_会社説明資料
livesense
PRO
0
9.1k
株式会社リブセンス会社紹介資料 / Invent the next common.
livesense
PRO
1
28k
26新卒_Webエンジニア職採用_会社説明資料
livesense
PRO
1
12k
中途セールス職_会社説明資料
livesense
PRO
0
250
EM候補者向け転職会議説明資料
livesense
PRO
0
120
Other Decks in Technology
See All in Technology
Reach American Airlines®️ Instantly: 19 Calling Methods for Fast Support in the USA
flyamerican
1
180
CDKコード品質UP!ナイスな自作コンストラクタを作るための便利インターフェース
harukasakihara
2
130
AWS認定を取る中で感じたこと
siromi
1
210
AWS CDK 開発を成功に導くトラブルシューティングガイド
wandora58
3
130
OpenTelemetryセマンティック規約の恩恵とMackerel APMにおける活用例 / SRE NEXT 2025
mackerelio
2
790
CDK Toolkit Libraryにおけるテストの考え方
smt7174
0
120
american aa airlines®️ USA Contact Numbers: Complete 2025 Support Guide
aaguide
0
390
【あのMCPって、どんな処理してるの?】 AWS CDKでの開発で便利なAWS MCP Servers特集
yoshimi0227
4
290
マネジメントって難しい、けどおもしろい / Management is tough, but fun! #em_findy
ar_tama
7
1.2k
SEQUENCE object comparison - db tech showcase 2025 LT2
nori_shinoda
0
180
LLM時代の検索
shibuiwilliam
2
430
MobileActOsaka_250704.pdf
akaitadaaki
0
170
Featured
See All Featured
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
46
9.6k
GitHub's CSS Performance
jonrohan
1031
460k
Intergalactic Javascript Robots from Outer Space
tanoku
271
27k
KATA
mclloyd
30
14k
It's Worth the Effort
3n
185
28k
BBQ
matthewcrist
89
9.7k
Optimising Largest Contentful Paint
csswizardry
37
3.3k
A Tale of Four Properties
chriscoyier
160
23k
YesSQL, Process and Tooling at Scale
rocio
173
14k
Speed Design
sergeychernyshev
32
1k
We Have a Design System, Now What?
morganepeng
53
7.7k
Building a Scalable Design System with Sketch
lauravandoore
462
33k
Transcript
Gerrit Takuya Shiotari
コードレビューとは ソフトウェア開発工程で見過ごされた誤りを検出・ 修正するためにソースコードの体系的な検査を行 うこと
良いコードを書くための5つの習慣 ▪ 読む ▪ 書く ▪ 道具を磨く ▪ 知る ▪
聞く コードレビュー
コードレビューの勧め 【コードレビューの必要性】 ▪ 正しく機能を作りこんだか を確認して、テスト中毒か ら脱却する ▪ 読みやすさ、保守性が高 いコードを書いているかを 確認して、デバッグ・保守・
エンハンスのコストを抑え る
コードレビューのメリット • 早期バグ発見 • リファクタリング • レビュアー/レビュイーのスキルアップ • コーディング規約の強制 •
新人育成 • システムの引き継ぎを容易にする
効率良くコードレビューを出来ていますか?
現状感じている問題点 • デプロイ後のコードをレビューでは後の祭り • レビュアーがレビュイーにどのバージョンのどの ファイルのどの行番号っていうのを伝えるのは 手間 • どのバージョンがレビュー済みかわかりにくい
Gerrit
Gerritとは • Google社員が開発している • Androidのオープンソースプロジェクトで利用さ れている • rietveld(オープンソース版 Mondrian)の後継? •
名前の由来はオランダの建築家ヘリット・トーマ ス・リートフェルト(Gerrit Thomas Rietveld)から 来ている
Gerritの特徴 • Webベースのコードレビューシステム • Gitを使ったプロジェクトのオンラインコードレビュー • 変更を並べて表示してレビューしやすくしている • レビュアーはインラインにコメントを追加することができる •
レビュイーはコミットをあるブランチ(refs/for/master)に PUSHして、レビュアーがレビューして承認すると自動的に masterブランチにマージされる
DEMO
考察 • レビューそのものを資産として残すことができる • レビューで承認されないとマスターにマージでき ないので、致命的なバグを未然に防ぐことがで きる • Jenkinsとの連携もいい感じ •
gitの使い方に慣れてないと難しい • ドキュメントが少ない
次週のテーマ ▪ AngularJS ▪ factor
Mondorian Pythonの開発者Guido Van Rossumが開発 Googleが採用しているSCMのPerfoce用
rietveld • オープンソース版Mondorian • 開発者はMondorianと同じくGuido Van Rossum • SCMにSubversion
Crucible • ConfluenceでおなじみAtlassian製 • http://www.atlassian. com/ja//software/crucible/overview
宍道湖(しんじこ) • Ruby on Rails製 • Google App Engine上で動作 •
rietveldのクローン • http://code.google.com/p/shinjiko/