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
37
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.
株式会社リブセンス 会社説明資料(報道関係者様向け)
livesense
PRO
0
1k
データ基盤の負債解消のためのリプレイス
livesense
PRO
0
250
26新卒_総合職採用_会社説明資料
livesense
PRO
0
4.5k
株式会社リブセンス会社紹介資料 / Invent the next common.
livesense
PRO
1
13k
26新卒_Webエンジニア職採用_会社説明資料
livesense
PRO
1
8.1k
中途セールス職_会社説明資料
livesense
PRO
0
200
EM候補者向け転職会議説明資料
livesense
PRO
0
86
コロナで失われたノベルティ作成ノウハウを復活させた話
livesense
PRO
0
210
転職会議でGPT-3を活用した企業口コミ要約機能をリリースした話
livesense
PRO
0
1.3k
Other Decks in Technology
See All in Technology
IAMのマニアックな話2025
nrinetcom
PRO
1
230
php-conference-nagoya-2025
fuwasegu
0
150
RemoveだらけのPHPUnit 12に備えよう
cocoeyes02
0
270
サイト信頼性エンジニアリングとAmazon Web Services / SRE and AWS
ymotongpoo
7
1.5k
Share my, our lessons from the road to re:Invent
naospon
0
140
偏光画像処理ライブラリを作った話
elerac
1
170
MIMEと文字コードの闇
hirachan
2
1.4k
OSS構成管理ツールCMDBuildを使ったAWSリソース管理の自動化
satorufunai
0
630
Visualize, Visualize, Visualize and rclone
tomoaki0705
9
82k
AIエージェント元年@日本生成AIユーザ会
shukob
1
200
Exadata Database Service on Cloud@Customer セキュリティ、ネットワーク、および管理について
oracle4engineer
PRO
2
1.5k
1行のコードから社会課題の解決へ: EMの探究、事業・技術・組織を紡ぐ実践知 / EM Conf 2025
9ma3r
10
3.7k
Featured
See All Featured
The Psychology of Web Performance [Beyond Tellerrand 2023]
tammyeverts
46
2.3k
Intergalactic Javascript Robots from Outer Space
tanoku
270
27k
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
marktimemedia
30
2.2k
How to Create Impact in a Changing Tech Landscape [PerfNow 2023]
tammyeverts
49
2.3k
Easily Structure & Communicate Ideas using Wireframe
afnizarnur
193
16k
How GitHub (no longer) Works
holman
314
140k
StorybookのUI Testing Handbookを読んだ
zakiyama
28
5.5k
How to Think Like a Performance Engineer
csswizardry
22
1.4k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
251
21k
Agile that works and the tools we love
rasmusluckow
328
21k
The Pragmatic Product Professional
lauravandoore
32
6.4k
Build The Right Thing And Hit Your Dates
maggiecrowley
34
2.5k
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/