Submit Search
Upload
コードレビューをより良くする Danger x Android
•
Download as PPTX, PDF
•
1 like
•
3,684 views
Toshiyuki Hirata
Follow
Android Test Night #1 9/21(木)の発表資料
Read less
Read more
Technology
Report
Share
Report
Share
1 of 13
Download now
Recommended
fastlane snapshotの並列実行についてまとめてみた
fastlane snapshotの並列実行についてまとめてみた
Toshiyuki Hirata
Android e2e testing at mercari
Android e2e testing at mercari
Vishal Banthia
fastlane x iOSアプリのCI
fastlane x iOSアプリのCI
Toshiyuki Hirata
DeNAにおけるSWETの役割
DeNAにおけるSWETの役割
Toshiyuki Hirata
2017年のiOSアプリ開発におけるCI事情
2017年のiOSアプリ開発におけるCI事情
Toshiyuki Hirata
iOSアプリの自動テストをはじめよう
iOSアプリの自動テストをはじめよう
Toshiyuki Hirata
バージョンアップ対応を軽減するサービス:マスティフ
バージョンアップ対応を軽減するサービス:マスティフ
Toshiyuki Hirata
iOSにおけるコードレビューを一歩先へ進める
iOSにおけるコードレビューを一歩先へ進める
Shunsuke Maeda
Recommended
fastlane snapshotの並列実行についてまとめてみた
fastlane snapshotの並列実行についてまとめてみた
Toshiyuki Hirata
Android e2e testing at mercari
Android e2e testing at mercari
Vishal Banthia
fastlane x iOSアプリのCI
fastlane x iOSアプリのCI
Toshiyuki Hirata
DeNAにおけるSWETの役割
DeNAにおけるSWETの役割
Toshiyuki Hirata
2017年のiOSアプリ開発におけるCI事情
2017年のiOSアプリ開発におけるCI事情
Toshiyuki Hirata
iOSアプリの自動テストをはじめよう
iOSアプリの自動テストをはじめよう
Toshiyuki Hirata
バージョンアップ対応を軽減するサービス:マスティフ
バージョンアップ対応を軽減するサービス:マスティフ
Toshiyuki Hirata
iOSにおけるコードレビューを一歩先へ進める
iOSにおけるコードレビューを一歩先へ進める
Shunsuke Maeda
Androidのテストを効率的にするために考えたこと
Androidのテストを効率的にするために考えたこと
Ataru Osaka
Dangerでpull requestレビューの指摘事項を減らす
Dangerでpull requestレビューの指摘事項を減らす
Shunsuke Maeda
Bluepillを使ったiOS自動テストの並列化
Bluepillを使ったiOS自動テストの並列化
Toshiyuki Hirata
UIテストの実行時間の短縮の方法
UIテストの実行時間の短縮の方法
Toshiyuki Hirata
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
Toshiyuki Hirata
Pull request時の画面差分取得の自動化
Pull request時の画面差分取得の自動化
Shunsuke Maeda
Xcodeの管理を楽に - Jenkins編 -
Xcodeの管理を楽に - Jenkins編 -
Toshiyuki Hirata
iOSアプリ開発のCI環境 - Jenkins編 -
iOSアプリ開発のCI環境 - Jenkins編 -
Toshiyuki Hirata
Androidアプリ開発のテスト環境
Androidアプリ開発のテスト環境
Toshiyuki Hirata
Xcode10での テスト周りの進化をふりかえる
Xcode10での テスト周りの進化をふりかえる
Toshiyuki Hirata
バージョンアップの対応を軽減するためのサービスの構築
バージョンアップの対応を軽減するためのサービスの構築
Toshiyuki Hirata
DroidKaigi_devicefarm
DroidKaigi_devicefarm
Shunsuke Maeda
iOSアプリにおけるリリースフローとCI環境
iOSアプリにおけるリリースフローとCI環境
Toshiyuki Hirata
スマホアプリディレクターが考えていること
スマホアプリディレクターが考えていること
Kazuaki KURIU
iOSで利用できるデバイスファームのメリット・デメリットの紹介
iOSで利用できるデバイスファームのメリット・デメリットの紹介
Shunsuke Maeda
JaSST'16 Tokyo モバイルセッション
JaSST'16 Tokyo モバイルセッション
mirer
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
Toshiyuki Hirata
20151021 cookpad talk_test_engineer
20151021 cookpad talk_test_engineer
Kazuaki Matsuo
5minQues - SWET近況報告
5minQues - SWET近況報告
Masaki Nakagawa
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
Kouhei Sutou
STFとAppiumをもちいたAndroidアプリの自動テスト
STFとAppiumをもちいたAndroidアプリの自動テスト
Toshiyuki Hirata
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
Naoki Umehara
More Related Content
What's hot
Androidのテストを効率的にするために考えたこと
Androidのテストを効率的にするために考えたこと
Ataru Osaka
Dangerでpull requestレビューの指摘事項を減らす
Dangerでpull requestレビューの指摘事項を減らす
Shunsuke Maeda
Bluepillを使ったiOS自動テストの並列化
Bluepillを使ったiOS自動テストの並列化
Toshiyuki Hirata
UIテストの実行時間の短縮の方法
UIテストの実行時間の短縮の方法
Toshiyuki Hirata
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
Toshiyuki Hirata
Pull request時の画面差分取得の自動化
Pull request時の画面差分取得の自動化
Shunsuke Maeda
Xcodeの管理を楽に - Jenkins編 -
Xcodeの管理を楽に - Jenkins編 -
Toshiyuki Hirata
iOSアプリ開発のCI環境 - Jenkins編 -
iOSアプリ開発のCI環境 - Jenkins編 -
Toshiyuki Hirata
Androidアプリ開発のテスト環境
Androidアプリ開発のテスト環境
Toshiyuki Hirata
Xcode10での テスト周りの進化をふりかえる
Xcode10での テスト周りの進化をふりかえる
Toshiyuki Hirata
バージョンアップの対応を軽減するためのサービスの構築
バージョンアップの対応を軽減するためのサービスの構築
Toshiyuki Hirata
DroidKaigi_devicefarm
DroidKaigi_devicefarm
Shunsuke Maeda
iOSアプリにおけるリリースフローとCI環境
iOSアプリにおけるリリースフローとCI環境
Toshiyuki Hirata
スマホアプリディレクターが考えていること
スマホアプリディレクターが考えていること
Kazuaki KURIU
iOSで利用できるデバイスファームのメリット・デメリットの紹介
iOSで利用できるデバイスファームのメリット・デメリットの紹介
Shunsuke Maeda
JaSST'16 Tokyo モバイルセッション
JaSST'16 Tokyo モバイルセッション
mirer
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
Toshiyuki Hirata
20151021 cookpad talk_test_engineer
20151021 cookpad talk_test_engineer
Kazuaki Matsuo
5minQues - SWET近況報告
5minQues - SWET近況報告
Masaki Nakagawa
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
Kouhei Sutou
What's hot
(20)
Androidのテストを効率的にするために考えたこと
Androidのテストを効率的にするために考えたこと
Dangerでpull requestレビューの指摘事項を減らす
Dangerでpull requestレビューの指摘事項を減らす
Bluepillを使ったiOS自動テストの並列化
Bluepillを使ったiOS自動テストの並列化
UIテストの実行時間の短縮の方法
UIテストの実行時間の短縮の方法
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
マスティフ (for Xcode beta) - バージョンアップ対応を軽減するためのサービス
Pull request時の画面差分取得の自動化
Pull request時の画面差分取得の自動化
Xcodeの管理を楽に - Jenkins編 -
Xcodeの管理を楽に - Jenkins編 -
iOSアプリ開発のCI環境 - Jenkins編 -
iOSアプリ開発のCI環境 - Jenkins編 -
Androidアプリ開発のテスト環境
Androidアプリ開発のテスト環境
Xcode10での テスト周りの進化をふりかえる
Xcode10での テスト周りの進化をふりかえる
バージョンアップの対応を軽減するためのサービスの構築
バージョンアップの対応を軽減するためのサービスの構築
DroidKaigi_devicefarm
DroidKaigi_devicefarm
iOSアプリにおけるリリースフローとCI環境
iOSアプリにおけるリリースフローとCI環境
スマホアプリディレクターが考えていること
スマホアプリディレクターが考えていること
iOSで利用できるデバイスファームのメリット・デメリットの紹介
iOSで利用できるデバイスファームのメリット・デメリットの紹介
JaSST'16 Tokyo モバイルセッション
JaSST'16 Tokyo モバイルセッション
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
iOSアプリ開発のテスト環境 - テストをはじめる最初の一歩 -
20151021 cookpad talk_test_engineer
20151021 cookpad talk_test_engineer
5minQues - SWET近況報告
5minQues - SWET近況報告
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
SEゼミ2015 - OSS Hack 4 Beginners - フィードバック
Similar to コードレビューをより良くする Danger x Android
STFとAppiumをもちいたAndroidアプリの自動テスト
STFとAppiumをもちいたAndroidアプリの自動テスト
Toshiyuki Hirata
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
Naoki Umehara
【XDev】A-2 アジリティ向上のためのツール活用
【XDev】A-2 アジリティ向上のためのツール活用
智治 長沢
はてなにおける Android アプリのソフトウェアテスト
はてなにおける Android アプリのソフトウェアテスト
Yu Nobuoka
ぼくのかんがえた iOSテスト戦略
ぼくのかんがえた iOSテスト戦略
Naoki Umehara
【JaSST'11 Kansai】 開発者とテスト担当者に最適なコラボレーションと効率化を!
【JaSST'11 Kansai】 開発者とテスト担当者に最適なコラボレーションと効率化を!
智治 長沢
Android studio で行ってみよう!!
Android studio で行ってみよう!!
Kazuaki Ueda
Team Foundation Server ~ 今を生きるエンジニアのための開発基盤とは 【BPStudy #63】
Team Foundation Server ~ 今を生きるエンジニアのための開発基盤とは 【BPStudy #63】
智治 長沢
The Twelve-Factor (A|M)pp with C#
The Twelve-Factor (A|M)pp with C#
Yuta Matsumura
How to implement CI with OutSystems using Jenkins and BDD Framework
How to implement CI with OutSystems using Jenkins and BDD Framework
YutoMiyazaki
Jenkins ユーザ・カンファレンス 2012 東京 S406-4/マルチステージ型継続的インテグレーションのすすめ
Jenkins ユーザ・カンファレンス 2012 東京 S406-4/マルチステージ型継続的インテグレーションのすすめ
atsushi_tmx
Web制作者視点で理解するソフトェアテスト
Web制作者視点で理解するソフトェアテスト
祐磨 堀
「最強」のチームを「造る」技術基盤 ディレクターズ・カット
「最強」のチームを「造る」技術基盤 ディレクターズ・カット
Rakuten Group, Inc.
Ldd13 present
Ldd13 present
Masashi Kayahara
デブサミ2013【15-C-8】セキュリティ要求仕様モデルプランで日本は変わるか?(百瀬昌幸氏)
デブサミ2013【15-C-8】セキュリティ要求仕様モデルプランで日本は変わるか?(百瀬昌幸氏)
Developers Summit
Visual Studio App Centerを公式サンプルアプリから学ぼうiOS(swift),Android(java)
Visual Studio App Centerを公式サンプルアプリから学ぼうiOS(swift),Android(java)
Shinya Nakajima
ここが良かったDatadog
ここが良かったDatadog
tyamane
Docker Enterprise Editionで実践するCaaS
Docker Enterprise Editionで実践するCaaS
DevOps Hub
なぜあなたのプロジェクトのDevSecOpsは形骸化するのか(CloudNative Security Conference 2022)
なぜあなたのプロジェクトのDevSecOpsは形骸化するのか(CloudNative Security Conference 2022)
Masaya Tahara
ワンクリックデプロイ101 #ocdeploy
ワンクリックデプロイ101 #ocdeploy
Ryutaro YOSHIBA
Similar to コードレビューをより良くする Danger x Android
(20)
STFとAppiumをもちいたAndroidアプリの自動テスト
STFとAppiumをもちいたAndroidアプリの自動テスト
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
Ricoh UCS for iPad でみる エンタープライズ アジャイル開発
【XDev】A-2 アジリティ向上のためのツール活用
【XDev】A-2 アジリティ向上のためのツール活用
はてなにおける Android アプリのソフトウェアテスト
はてなにおける Android アプリのソフトウェアテスト
ぼくのかんがえた iOSテスト戦略
ぼくのかんがえた iOSテスト戦略
【JaSST'11 Kansai】 開発者とテスト担当者に最適なコラボレーションと効率化を!
【JaSST'11 Kansai】 開発者とテスト担当者に最適なコラボレーションと効率化を!
Android studio で行ってみよう!!
Android studio で行ってみよう!!
Team Foundation Server ~ 今を生きるエンジニアのための開発基盤とは 【BPStudy #63】
Team Foundation Server ~ 今を生きるエンジニアのための開発基盤とは 【BPStudy #63】
The Twelve-Factor (A|M)pp with C#
The Twelve-Factor (A|M)pp with C#
How to implement CI with OutSystems using Jenkins and BDD Framework
How to implement CI with OutSystems using Jenkins and BDD Framework
Jenkins ユーザ・カンファレンス 2012 東京 S406-4/マルチステージ型継続的インテグレーションのすすめ
Jenkins ユーザ・カンファレンス 2012 東京 S406-4/マルチステージ型継続的インテグレーションのすすめ
Web制作者視点で理解するソフトェアテスト
Web制作者視点で理解するソフトェアテスト
「最強」のチームを「造る」技術基盤 ディレクターズ・カット
「最強」のチームを「造る」技術基盤 ディレクターズ・カット
Ldd13 present
Ldd13 present
デブサミ2013【15-C-8】セキュリティ要求仕様モデルプランで日本は変わるか?(百瀬昌幸氏)
デブサミ2013【15-C-8】セキュリティ要求仕様モデルプランで日本は変わるか?(百瀬昌幸氏)
Visual Studio App Centerを公式サンプルアプリから学ぼうiOS(swift),Android(java)
Visual Studio App Centerを公式サンプルアプリから学ぼうiOS(swift),Android(java)
ここが良かったDatadog
ここが良かったDatadog
Docker Enterprise Editionで実践するCaaS
Docker Enterprise Editionで実践するCaaS
なぜあなたのプロジェクトのDevSecOpsは形骸化するのか(CloudNative Security Conference 2022)
なぜあなたのプロジェクトのDevSecOpsは形骸化するのか(CloudNative Security Conference 2022)
ワンクリックデプロイ101 #ocdeploy
ワンクリックデプロイ101 #ocdeploy
More from Toshiyuki Hirata
iOSにおけるパフォーマンス計測
iOSにおけるパフォーマンス計測
Toshiyuki Hirata
iOSにおけるパフォーマンス計測
iOSにおけるパフォーマンス計測
Toshiyuki Hirata
iOSにおける自動テストの並列化
iOSにおける自動テストの並列化
Toshiyuki Hirata
UIテスト(Espresso)の高速化をさらにすすめる
UIテスト(Espresso)の高速化をさらにすすめる
Toshiyuki Hirata
UIテストの実行時間を短縮させる方法
UIテストの実行時間を短縮させる方法
Toshiyuki Hirata
我が家のLINEを中心とした情報共有
我が家のLINEを中心とした情報共有
Toshiyuki Hirata
レビューのコストを削減するための施策
レビューのコストを削減するための施策
Toshiyuki Hirata
More from Toshiyuki Hirata
(7)
iOSにおけるパフォーマンス計測
iOSにおけるパフォーマンス計測
iOSにおけるパフォーマンス計測
iOSにおけるパフォーマンス計測
iOSにおける自動テストの並列化
iOSにおける自動テストの並列化
UIテスト(Espresso)の高速化をさらにすすめる
UIテスト(Espresso)の高速化をさらにすすめる
UIテストの実行時間を短縮させる方法
UIテストの実行時間を短縮させる方法
我が家のLINEを中心とした情報共有
我が家のLINEを中心とした情報共有
レビューのコストを削減するための施策
レビューのコストを削減するための施策
Recently uploaded
[DevOpsDays Tokyo 2024] 〜デジタルとアナログのはざまに〜 スマートビルディング爆速開発を支える 自動化テスト戦略
[DevOpsDays Tokyo 2024] 〜デジタルとアナログのはざまに〜 スマートビルディング爆速開発を支える 自動化テスト戦略
Ryo Sasaki
論文紹介:Content-Aware Token Sharing for Efficient Semantic Segmentation With Vis...
論文紹介:Content-Aware Token Sharing for Efficient Semantic Segmentation With Vis...
Toru Tamaki
Open Source UN-Conference 2024 Kawagoe - 独自OS「DaisyOS GB」の紹介
Open Source UN-Conference 2024 Kawagoe - 独自OS「DaisyOS GB」の紹介
Yuma Ohgami
TSAL operation mechanism and circuit diagram.pdf
TSAL operation mechanism and circuit diagram.pdf
taisei2219
論文紹介:Automated Classification of Model Errors on ImageNet
論文紹介:Automated Classification of Model Errors on ImageNet
Toru Tamaki
Postman LT Fukuoka_Quick Prototype_By Daniel
Postman LT Fukuoka_Quick Prototype_By Daniel
danielhu54
SOPを理解する 2024/04/19 の勉強会で発表されたものです
SOPを理解する 2024/04/19 の勉強会で発表されたものです
iPride Co., Ltd.
スマートフォンを用いた新生児あやし動作の教示システム
スマートフォンを用いた新生児あやし動作の教示システム
sugiuralab
論文紹介:Semantic segmentation using Vision Transformers: A survey
論文紹介:Semantic segmentation using Vision Transformers: A survey
Toru Tamaki
Recently uploaded
(9)
[DevOpsDays Tokyo 2024] 〜デジタルとアナログのはざまに〜 スマートビルディング爆速開発を支える 自動化テスト戦略
[DevOpsDays Tokyo 2024] 〜デジタルとアナログのはざまに〜 スマートビルディング爆速開発を支える 自動化テスト戦略
論文紹介:Content-Aware Token Sharing for Efficient Semantic Segmentation With Vis...
論文紹介:Content-Aware Token Sharing for Efficient Semantic Segmentation With Vis...
Open Source UN-Conference 2024 Kawagoe - 独自OS「DaisyOS GB」の紹介
Open Source UN-Conference 2024 Kawagoe - 独自OS「DaisyOS GB」の紹介
TSAL operation mechanism and circuit diagram.pdf
TSAL operation mechanism and circuit diagram.pdf
論文紹介:Automated Classification of Model Errors on ImageNet
論文紹介:Automated Classification of Model Errors on ImageNet
Postman LT Fukuoka_Quick Prototype_By Daniel
Postman LT Fukuoka_Quick Prototype_By Daniel
SOPを理解する 2024/04/19 の勉強会で発表されたものです
SOPを理解する 2024/04/19 の勉強会で発表されたものです
スマートフォンを用いた新生児あやし動作の教示システム
スマートフォンを用いた新生児あやし動作の教示システム
論文紹介:Semantic segmentation using Vision Transformers: A survey
論文紹介:Semantic segmentation using Vision Transformers: A survey
コードレビューをより良くする Danger x Android
1.
コードレビューをより良くする Danger x Android 2017/09/21(Thu) Android
Test Night #1 平田敏之(@tarappo)
2.
自己紹介 平田 敏之@DeNA /
SWET (Software Engineer in Test) ミッション DeNAサービス全般の品質向上 DeNAエンジニアの開発生産性向上 私がやっていること(の一部) クライアントアプリ周りのテスト戦略、自動テストの開発 CI/CD環境整備 テスト基盤環境の開発(マスティフ、ダルメシアン etc) iOS / Android Test Night 主宰
3.
SWETのBLOGはじめました http://swet.dena.com/ DeNA Testing Blog
4.
Danger http://danger.systems/ • Dangerとは? (
js / ruby ) • PRの(一次)チェックをしてくれる • チェックするルールは自ら決める(プロジェクト次第) • プラグインの作成も出来る • すでに世にあるプラグインを使えば導入が簡単 • 導入事例も増えてきている • http://techlife.cookpad.com/entry/2017/06/28/190000 • https://techblog.recruitjobs.net/development/danger_driven_development
5.
コードレビュー コードレビューに時間がかかっていませんか? • 本筋じゃない箇所への指摘 • チケットのリンクが貼られていない •
PRの向き先が正しくない • コードフォーマッターに任せられるような指摘 etc • UI変更時のチェック • PRにスクリーンショットを貼ってもらったり.. • 変更したコードの動作確認をしているかどうか • CI(など)で動かしているかどうか 過ぎていく時間 人間によるレビューは貴重なので自動化出来ることは自動化させよう
6.
Dangerでのチェック例 • PRの状態チェック • ラベルやタイトルにWIPがついていないかどうか •
github.pr.titleやgithub.pr_labelsでチェック可能 • PRの本文チェック • 対応するチケットが書かれているか • github.pr_bodyでチェック可能 https://github.com/Moya/Moya/blob/master/Dangerfile より
7.
Danger Plugin 色々あります • SWET製 •
CONFLICT CHECKER • SLACK http://danger.systems/ruby より
8.
チームのルール Androidアプリ 共通(独自Plugin化) • WIPとタイトルやラベルに記載していないこと • PRの説明文にチケットのURLがあること •
PRに記載されているタスク全てにチェックがついていること Android独自(独自Plugin化) • プロダクトコードを変更した場合 • アプリのビルド、自動テストをおこなっているか • テストコードを変更した場合 • 自動テストをおこなっているか • リソースを変更した場合 • スクリーンショット取得をおこなっているか
9.
コード変更時のチェックルール • プロダクトコードを変更した場合 • ビルド、テストを実行する必要あり •
ビルド・テストが成功(1) • (1)から最新のコミットまでの間にプロダクトコードに変更なけれ ば再度実行する必要はない • テストコードを変更した場合 • テストを実行する必要あり
10.
UI変更時のチェックルール • リソースを変更した場合 • 各画面のスクリーンショットをPRにいれこむ •
UIテスト(Spoon)を用いて各画面のスクリーンショットを取得 • githubの機能で前回の画像と比較できる
11.
レビューまでのフロー • PRを出す(WIPでもOK) • 設計相談をする場合は早めに出す •
実装が完了 • CIで動かすべきものは動かしておく • PR上でreviewとコメント • ルールを全てクリアしていたら • メンバーにレビュー依頼が飛ぶ
12.
ルールをクリアしなかった場合 実際の指摘例 指摘理由 • WIPとタイトルに記載されている • PRの本文にチケットのURLが無い •
ソースコード変更後にアプリのビルドをおこ なっていない • テストコード変更後にテストの実行をおこな っていない
13.
まとめ • コードレビュー前の指摘(の一部)は自動化しよう その結果 • 実装者(レビューイー) •
何をするべきかルール化されている、間違えれば指摘されるので分かりやすい • レビュアー • (多少の)安心感をもってコードレビューができる • 本質的な箇所をコードレビューできるようになる
Download now