Bitbucketを活用したコードレビュー改善事例

ULS
Copyright © 2011-2016 UL Systems, Inc. All rights reserved.
Proprietary & Confidential Powered by
Bitbucket を活用したコードレビュー改善事例
2016/09/27
ウルシステムズ株式会社
伊藤昂祐

ULS
Proprietary & Confidential Powered by 1
自己紹介
いとうこうすけ
伊藤昂祐
kosuke.ito@ulsystems.co.jp
ウルシステムズ株式会社
https://www.ulsystems.co.jp/
インターネット広告事業会社の開発支援中
社内では、アジャイル推進活動中
認定スクラムマスタ
スノボとスープカレーを愛する

ULS
アジェンダ
1. プロジェクト紹介とインターネット広告の進歩について
2. Bitbucket を活用したコードレビュー改善事例

ULS
プロジェクト紹介とインターネット広告の
進歩について

ULS
プロジェクト紹介
クライアント
– ソネット・メディア・ネットワークス株式会社様
主な事業内容
– DSP事業
Logicad を提供
RTBを活用した広告配信最適化
機械学習・人工知能を活用したDMPによる広告配信精度の向上
プロジェクト支援概要
– DSP/RTBシステム開発支援
– 使用しているAtlassian製品は Confluence, Jira, Bitbucket

ULS
いろいろなインターネット広告
バナー広告
検索連動型
広告
アプリ広告
動画広告
ネイティブ広告

ULS
インターネット広告の進歩
Webサイト運営者広告主
 商品を世の中に広めたい
 多くの見込み客に広告を出
したい
 自社メディア(Webサイト)
で収益をあげたい
広告業界における双方の欲求

ULS
純広告
AD
広告代理店
Webサイト運営者
広告主
 広告主が期間・Webサイトを選び、広告を掲載する

ULS
アドネットワーク
広告代理店広告主
 多数のWebサイト上で広告配信する仕組み
 Webサイトの特性に合わせたターゲティングも可能
AD
AD
AD

ULS
アドエクスチェンジ
 アドネットワーク同士で広告枠を融通する
AD
AD
AD
AD

ULS
SSP・DSP・RTB
 SSP(Supply Side Platform) – 広告枠の価値を最大化
 DSP(Demand Side Platform) – 広告主の利益を最大化
 RTB(Real Time Bidding) – リアルタイムで広告の競争入札する仕組み
AD
アドネットワークAD
AD
AD
AD
AD
DSPSSP
RTB

ULS
RTBの流れ
AD
ID=XYZ SSP事業者
DSP3
DSP2
DSP１
WebサイトA
WebサイトA に、XYZさんが来ました。
誰か広告を出したい方いませんか？
1円
2円
3円
WIN
PVから広告表示まで 0.1秒(100ミリ秒)で広告オークショ
ンが行われる

ULS
DMP
AD
アドネットワークAD
AD
AD
AD
AD
DSPSSP
RTB
 データマネジメントプラットフォーム
 オーディエンスデータを蓄積・分析して広告配信に活用
DMP

ULS
DSP/RTB事業のシステム開発を行うにあたって
業界は急速に進歩していて、新機能を素早く開発することが
求められている
広告配信は24時間365日行われているため、可用性が高く高
品質なシステムが求められている
RTB入札の性能要求が厳しく、継続的な性能改善が求められ
る

ULS
Bitbucket を活用した
コードレビュー改善事例

ULS
コードレビューの目的は、ただコードの誤りを修正
するだけではありません。重要なのは、チーム全員
に同じ知識を共有させること、またコーディングに
おいて全員が守るべきガイドラインを確立すること
です。コードレビューを成功させるために最も有効
な方法は、レビューを楽しいものにすることです。
by Mattias Karlsson 「プログラマが知るべき97のこと」より

ULS
プロジェクトの前提
背景
– SVN から Git へ移行を行い、その後の運用についての話です
リポジトリ管理
– Bitbucket(Git) を使用
チーム
– チームは 5±2人で運用
– チームの Git 経験者はゼロに近い状態
– リリース周期は 1週間～1ヶ月
– アジャイルなチームを目指したフラットな組織

ULS
ブランチ戦略
要求
– プロダクトの安定稼動が出来ること
– 新規機能の開発を活発に行えること
– バグ発生時にバージョンを容易に戻せること
– 機能開発中にバグ修正を差し込めること
※ A Successful Git Branching Model より
http://nvie.com/posts/a-successful-git-
branching-model/
git-flow の採用
– 相反する要求を満たすブランチの運用
– メジャーリリース・マイナーリリースが混在する
プロダクトに有効
– Github Flow や GitLab Flow と比較すると多少
複雑で慣れるまで時間がかかる

ULS
SouceTree とは
https://ja.atlassian.com/software/sourcetree
Atlassian 社が提供
Gitクライアントツール
コマンドライン不要。GUIでリポジトリの運用可能

ULS
SouceTree の運用
リポジトリ操作がボタンひとつで簡単に出来る
コミットツリーの参照が可能
「Git Flow」ボタンにより、git-flow
に従った操作を行うよう助ける

ULS
SouceTree を使って良かったこと
 コマンドを使用することなく Gitリポジトリの
操作が容易に出来る
 開発スピードを落とすことなく、git-flow の
導入と学習を進めることが出来る

ULS
リポジトリ管理
https://bitbucket.org/
Atlassian 社が提供
Gitリポジトリ管理ツール

ULS
当初のコードレビューのやり方
プルリクエ
ストの作成
開発者
対面レビュー
プルリクエス
トのマージ
承認
レビュワー
修正が必要か
コードの修正
必要
不要
開発者
説明
指摘

ULS
プルリクエストの作成
タイトル
説明
レビュアー
タイトル・説明の入力・依頼するレ
ビュアーの選択を行いプルリクエスト
を作成する

ULS
プルリクエストの作成
説明
レビュアー
タイトル
 プルリクエスト駆動の開発でコードレビュー
が必須に

ULS
コードレビューの実施
インラインディスカッション
– インラインコメント・メンションを使用した
コミュニケーション
– マークアップシンタックス
コードの近くにフィードバックを書
くことで、レビュー効率アップ
横並び差分表示も可能

ULS
コードレビューに関する情報の表示
JIRAの課題ワークフローにレビュー中のステータスを追加
タスクかんばんの列に「レビュー中」を
追加。レビューステータスを可視化
開発ブランチの状態をひとまとめ
に表示。すぐに状況の認識が可能

ULS
プルリクエストのマージ
マージ方法
マージ条件の設定
承認者の人数・タスクの解決・ビルドの成功
を条件に設定することが出来る。
→開発者の独断で判断でマージすることを防
止する。
ブランチパーミッション
特定のブランチに対して、削除・変更を防
ぐ設定を行うことが出来る。
→ git flow で開発者に許可されていない操
作を事前に防止する。

ULS
Bitbucket(Git)の開発作業に慣れてきたころ、
コードレビューにいくつかの問題が・・・

ULS
こんなコードレビューはイヤだ
何を確認すれば良いかわからない
クソコードが大量発生する
なかなかレビューされない
レビューのコストが高すぎる
コードの誤りを指摘するだけになっている

ULS
タイトル・説明の記載をルール化する
プルリクエストの改善
1. 対面レビューは必要か
• 要 or 不要を記載
• 重要度やコードの複雑度によって開発者が判断
• 直接説明したほうが、はやいものも要と判断
2. 緊急度
• 高中低 or レビュー期限をブランチ特性から記載
3. 対応内容
• プルリクエストの単位を小さくする
• プルリクエストの目的を明確にする
• レビューの観点を明確にする
• 関連リンクを記載
説明
1. WIP(Work In Progress)の活用
2. RFC(Request For Comment)の活用
タイトル

ULS
コードレビューの改善
対面レビューは必要なものだけに限定して行う
その代わりにインラインディスカッションの質を上げる
– 誤りの指摘を書くだけではなく、気づきや質問も記載するよう啓蒙
– 意見募集の書き込み
– 新しい実装方法を提案と「いいね！」の使用
– 修正・調査が必要なコメントにはタスクの作成

ULS
改善による変化
何を確認すれば良いかわからない
→プルリクエストの説明改善でレビューポイントが明確に
クソコードが大量発生する
→WIP・RFC でリスクを事前につぶせるように
なかなかレビューされない
→プルリクエストを小さくすることでレビューが容易に
レビューのコストが高すぎる
→必要最低限の対面レビューをすることコスト削減
コードの誤りを指摘するだけになっている
→気付きや質問を書き込むことで徐々に議論が活発に
→議論の中から、知識の共有や新しいアイデアが出るように

ULS
現在のコードレビューのやり方
プルリクエ
ストの作成
対面レビュー
が必要か
開発者
対面レビュー
机上レビュー
プルリクエス
トのマージ
承認
レビュワー
修正が必要か
コードの修正
必要
必要
不要
不要
開発者
チーム全体で双方向的
コミュニケーション
重要なものに限定して、
対面レビュー
レビューに必要な情報を明確に
して、円滑なレビュー実施へ

ULS
プラグイン連携
Slack notifications plugin
– Bitbucket のイベントを Slack に通知する
Webhook to Jenkins for Bitbucket
– Bitbucket のイベントを Jenkins に通知する
ビルド・デプロイ・
自動テストの実行
イベントの通知結果通知

ULS
Slack への通知
 応答性の向上
 チームが知るべき情報の通知
– 障害Alert
– チームの予定
– ビルドデプロイ・自動テストの結果
– タスクのイベントを通知

ULS
ありがとうございました

Bitbucketを活用したコードレビュー改善事例

Recommended

Recommended

More Related Content

What's hot

What's hot (20)

Similar to Bitbucketを活用したコードレビュー改善事例

Similar to Bitbucketを活用したコードレビュー改善事例 (20)

Bitbucketを活用したコードレビュー改善事例