Dangerでpull requestレビューの指摘事項を減らす

Copyright © DeNA Co.,Ltd. All Rights Reserved.
DangerでPull Request
レビューの指摘事項
を減らす
AKIBA.swift スーパー勉強会
April 17, 2017
前田隼輔
SWETグループ
DeNA Co., Ltd.

自己紹介
 前田隼輔
⁃ 所属
• DeNA システム本部 SWETグループテスト基盤チーム
⁃ 経歴
• ３月に DeNA SWETグループに Join
• それまでは SIer でWebアプリ開発
⁃ 普段の業務
• CI/CD環境整備、ツールの検証
• バージョンアップ自動検証システムの開発
2

Pull Requestでこんなことありませんか？
3
ソースコード
いじってるけど
ビルドは通った？
masterブランチ
にはマージしないで
ください
対応するチケットの
URLを載せてください
ロジックや設計以外での指摘事項が多すぎる

そこで
4
http://danger.systems/
https://github.com/danger/danger

5
🚫 Stop saying “you forgot to …”
in code review

今日お話しすること
 Dangerとは
 Dangerでできること
 SWET テスト基盤チームでの事例
6

 Dangerとは
7

Dangerとは
 Pull Request時の確認事項を自動化できるツール
 Dangerfile を作成し、その中で自由にチェックできる
 チェック結果はホスティングサービスに通知し、
Pull Request内の commit status に反映できる
 GitやGitホスティングサービスから情報を取得している
8

Dangerとは
 Pull Request時の確認事項を自動化できるツール
 Dangerfile を作成し、その中で自由にチェックできる
 チェック結果はホスティングサービスに通知し、
Pull Request内の commit status に反映できる
 GitやGitホスティングサービスから情報を取得している
9
Pull Requestの確認を自動化
するための基盤の提供

Dangerとは
10

 Dangerとは
11

Dangerでできること
 Dangerfile では Ruby構文が使える
 ホスティングサービスへの通知のためのDSL
⁃ message / warn / fail / markdown
 git や github といった変数に情報が含まれている
 github.api で OctoKit (GitHub APIの Rubyクライアント)を取得できる
12

13
Ruby と GitHub API
の知識があれば（なくてもググれば）
なんでもできる

14
目的を見失わないように
🚫 Stop saying “you forgot to …”
in code review

 Dangerとは
15

利用しているツールの組み合わせ
 Gitホスティングサービス
⁃ GitHub Enterprise
 CI / CDツール
⁃ Jenkins
⁃ + GitHub Pull Request Builder
 ビルドマネージャ
⁃ fastlane 🚀
16

GitHub Pull Request Builder + fastlane
 GitHub Pull Request Builder
⁃ PRのコメント(Phrase)でビルドを開始できる
 fastlane 🚀
⁃ モバイルアプリに便利なビルドマネージャ
lane名によって処理を振り分けることができる
17
${ghgrpCommentBody}
にPhraseが入る

GitHub Pull Request Builder + fastlane
 GitHub Pull Request Builder
⁃ commit status に phrase (= lane名) を指定
18
Contextを
Phraseにする

fastlaneには danger アクションが用意されている
bundle exec fastlane する場合は必ず
use_bundle_exec: false にしましょう
19
> fastlane actions danger

review レーンで Danger を実行
20

チームで用いているPull Requestルール
21
 .storyboard または .xibに変更があった場合、
画像が貼り付けられていること
 ソースディレクトリに変更があった場合、
ビルドとシミュレータによるテストが実行されていること
 テストディレクトリに変更があった場合、
シミュレータによるテストが実行されていること
 ビルドおよびテストが実行されていた場合、
最新の結果が成功していること
 Pull Request本文に PULL_REQUEST_TEMPLATE.md と同じ項目
があり、テンプレートから変更されていること
 TODO のチェックが埋まっていること
 ビルドとテストはラベルによってスキップ可能
 全てOK だった場合に In Review のラベルを追加

コード化
 .storyboard または .xibに変更があった場合、
PR本文中に画像が貼り付けられていること
 テストディレクトリ以下に変更があった場合、
シミュレータによるテストが実行されていること
 ビルドおよびテストが実行されていた場合、
最新の結果が成功していること

共通なものはプラグイン化
23
 Pull Request本文に PULL_REQUEST_TEMPLATE.md と同じ項目
があり、テンプレートから変更されていること
 TODO のチェックが埋まっていること
 プラグインの作成も簡単
> danger plugins create プラグイン名
で作られたプロジェクト（danger-プラグイン名）の
lib/プラグイン名/plugin.rb
を編集
 作成したプラグインの利用
Gemfileに追加するだけ
PR本文の形式は
プラグインでまとめて検証

ラベルを自動的に貼る
24
 全てOK だった場合に In Review のラベルを追加
ラベルがある場合は安心してレビューできる

最後に
25
 Danger を使って PRの形式や最低限の条件をチェックを自動化
 プロジェクトに Dangerfile を用意して
Pull Request レビュー（前）指摘事項を減らしましょう

Dangerでpull requestレビューの指摘事項を減らす

More Related Content

What's hot

Similar to Dangerでpull requestレビューの指摘事項を減らす

Dangerでpull requestレビューの指摘事項を減らす