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
260
26新卒_総合職採用_会社説明資料
livesense
PRO
0
4.6k
株式会社リブセンス会社紹介資料 / Invent the next common.
livesense
PRO
1
13k
26新卒_Webエンジニア職採用_会社説明資料
livesense
PRO
1
8.2k
中途セールス職_会社説明資料
livesense
PRO
0
200
EM候補者向け転職会議説明資料
livesense
PRO
0
87
コロナで失われたノベルティ作成ノウハウを復活させた話
livesense
PRO
0
210
転職会議でGPT-3を活用した企業口コミ要約機能をリリースした話
livesense
PRO
0
1.3k
Other Decks in Technology
See All in Technology
Amazon Q Developerの無料利用枠を使い倒してHello worldを表示させよう!
nrinetcom
PRO
2
120
Ruby on Railsで持続可能な開発を行うために取り組んでいること
am1157154
3
160
Oracle Database Technology Night #87-1 : Exadata Database Service on Exascale Infrastructure(ExaDB-XS)サービス詳細
oracle4engineer
PRO
1
190
アジャイルな開発チームでテスト戦略の話は誰がする? / Who Talks About Test Strategy?
ak1210
1
650
IAMポリシーのAllow/Denyについて、改めて理解する
smt7174
2
210
データベースの負荷を紐解く/untangle-the-database-load
emiki
2
530
30→150人のエンジニア組織拡大に伴うアジャイル文化を醸成する役割と取り組みの変化
nagata03
0
200
1行のコードから社会課題の解決へ: EMの探究、事業・技術・組織を紡ぐ実践知 / EM Conf 2025
9ma3r
12
4.2k
Pwned Labsのすゝめ
ken5scal
2
460
DeepSeekとは?何がいいの? - Databricksと学ぶDeepSeek! 〜これからのLLMに備えよ!〜
taka_aki
1
110
Autonomous Database Serverless 技術詳細 / adb-s_technical_detail_jp
oracle4engineer
PRO
17
45k
ExaDB-XSで利用されているExadata Exascaleについて
oracle4engineer
PRO
3
270
Featured
See All Featured
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
33
2.1k
Chrome DevTools: State of the Union 2024 - Debugging React & Beyond
addyosmani
4
380
Speed Design
sergeychernyshev
27
810
How To Stay Up To Date on Web Technology
chriscoyier
790
250k
Learning to Love Humans: Emotional Interface Design
aarron
273
40k
How to train your dragon (web standard)
notwaldorf
91
5.9k
Building Applications with DynamoDB
mza
93
6.2k
A Philosophy of Restraint
colly
203
16k
Fireside Chat
paigeccino
34
3.2k
How STYLIGHT went responsive
nonsquared
98
5.4k
Stop Working from a Prison Cell
hatefulcrawdad
267
20k
The Web Performance Landscape in 2024 [PerfNow 2024]
tammyeverts
4
430
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/