AbemaTV モバイルアプリの開発体制と開発プロセスの話

Editor's Notes

1. では早速開発体制についてですが、
2. ということで早速開発体制についてですが、
3. 現在AbemaTVの開発局は約50名程度います
4. えー図にするとこのような感じで、左からDirector, Server Design, Web, iOS, Android, QAといたチームに分かれていて、それぞれのチームの代表でBoardメンバーが構成されています.
5. 今日はこの中で、iOSチームについての開発体制や開発スタイルについてお話したいと思います。
6. まずiOSチームの中でもチームが分かれていて ビデオチーム、グロースチーム、本質改善チーム、テレビデバイスチームと大きく4つに分かれていて、その他にどれに属さない案件、プロジェクトなどがあったりします。
7. iOSチームですが、
8. 現在10名体制で開発しています。 立ち上げ初期は４名程度だったのでだいぶ大きなチームになりました。
9. はい、みんなに顔出しOKもらったので載せておきますw
10. で、具体的には、先程お話したビデオ、グロース、本質改善、テレビデバイスチームはこのような形で分かれていて、その他の案件はケースバイケースでその時の状況で調整して対応しています。
11. AbemaTVの対応デバイスはPC, iPhone/iPad, Android/タブレット、 AppleTV, AndroidTV, Google Castがあります。
12. この中でiOSチームはiPhone, iPad, AppleTV, あとGoogle Castはテレビ側のrecieverとクライアント側のsenderがありますが、sender側が担当範囲になります。
13. iOSチームが管理しているcodebaseですが、
14. 代表的なレポジトリはiosとtvosの２つになります あとは共通のapiモジュールや あっprotocolBuffer使っているのですが、protoのswiftのレポジトリ 共通のscript郡のcmdshelfレポジトリなどがあります
15. これはcodeのfile数や行数を出したものですが
16. iosで約9万行、tvosで27000行、apiで3800行くらいですね
17. こちらはiosレポジトリのGitHubのinsightsになります
18. iosのレポジトリだと、これは9月19~ 10月19日の1ヶ月で162個のpull requestがマージされていて、営業で1日平均すると、、だいたい7~8個のpull requestがマージされています、 ちなみに一番右の全然commitしてないのはデザイナーで画像差し替えとか文言変更はデザイナーが対応してくれたりします
19. こちらはtvOSですが、おもに２人で開発していて
20. tvosのレポジトリだと、57個のpull requestが1ヶ月でマージされていて、営業で1日平均すると、、だいたい2~3個のpull requestがマージされています
21. こちらは共通のapiモジュールのレポジトリですが
22. こちらもちょくちょく変更がはいります
23. 何が言いたいかといいますと、まぁみんな仕事していますとw もとい毎日大量のコードが変更されていることがわかると思います
24. では、そういった開発をしているフローはどうなっているかというと、
25. こちらの図のような感じで、基本はスクラム開発でsprintを２週間に区切って開発しています。
26. もうちょっと細かくした図ですが、これは少し前に実践していたフローですが、 開発を二週間、QAを１週間で2週間ごとの申請でやっていました。 ただこれでやっていると問題がありまして、
27. ここの期間ですね。
28. 開発とQA期間の重複がつらいですと。 QA中にバグ対応などがお主に発生すると、次のsprintへの影響がもろにでてしまうのでこれは無理があるということで、
29. 改善しました。
30. 開発期間1週間、QA期間1週間という定義でやることにしました。 今のところこれでスムーズにいっています。
31. 長い開発はこのような形でsprintを跨いだ開発になります
32. 厳密に１週間/１週間にはならないときも、もちろんあるので、QA期間中から開発をはじめているものもあります。計画ミーティングごとに何をやるか決めますが、ある程度先のsprintの開発まで決めているのでこういった調整が容易になっています。
33. ではそもそもタスクはどうやって発生しているのかということですが、
34. パターン1として、、、、、いわゆるトップダウン的なものですね
35. パターン2は、、、、、といったボトムアップでタスク化することも多々あります。
36. ではどうやってタスクを見積もっているかということですが、
37. 一応タスクにストーリーポイントを付けています。1ポイントが軽微なもの、2ポイントが0.5スプリント、3ポイントが1スプリント、またはそれ以上といった定義しかしていません
38. ものすごいおおざっぱですねw この数値の付け方はスクラム開発だと変な定義だと思いますが、 AbemaTVは事業の流れによって案件の優先度がけっこう変わるので、ガチガチにポイントつけてSprintを回すのは向いてなくて、一人ひとりが今どれくらい案件を抱えていて、どのくらい重い開発をしているかを見える化する程度にしています。 ただここももう少しちゃんとポイント定義して回せるようにしていきたいと考えています
39. タスクの優先度定義ですが、
40. チケットを見ただけでこれは今回マストで入れないといけないのか、次に回してもいいのかエンジニアが判断できるようにしています。こういった共通言語の定義があることで、ディレクターなどとの会話がスムーズにいくといった利点もあります。
41. あと会議体ですが、
42. 大きく4つありまして、スプリント計画を行うスプリント計画ミーティング、スプリントの振り返りを行うスプリントレビュー、メンバー各自の案件ごとのミーティングとiOSチームの週一の定例をやっています。毎日の行うようなデイリースクラムはやっていません
43. あと使っているツールの紹介します
44. 使っているツールですが、いわゆる一般的なものでチャットツールのSlack、チケット管理にJIRA, ドキュメントツールにConfluenceとesa、ソースコード管理にGitHub、CI用にBitriseと自動化用にJenkinsとcmdshelfなどを使っています。
45. cmdshlefはオリジナルなので知らない人も多いかと思うので少し紹介すると
46. リモートリポジトリの実行可能ファイルをローカルファイルのように統合して扱えるようにするもので、iOSチームのトッシーさんという方がつくっていてSwift製です
47. こちら実際の例になります。
48. 最初の方でお話しましたが、iOSチームで管理している共通script置き場のcmdshelf-iosというレポジトリがあります
49. cmdshelfをinstallしてこのレポジトリをaddして、cmdshelf listを実行すると実行できるコマンド郡が表示されるようになります
50. listで表示されたものをcmdshelf runで実行できるといった形で、リモートのレポジトリのscript郡を手軽に使えてけっこう便利なものになります。
51. はい、次に開発スタイルですが、
52. こちらはContributing.mdのPullRequestsについての抜粋ですが、、、 といったルールも定義しています
53. これはPull Requestsのリストの抜粋ですが、ちょっと多いとか少ないとかの平均はわかりませんが、自分の経験上、けっこう活発なレビュー文化だと思います。
54. こちらはコーディング規約でGoodな書き方とBadの書き方というのを明示して、なるべくチーム内でコードの統一性を出すようにしています。
55. こちらはテストの実行結果ですが、現在2009ケースのテストがあり、極力テストは書くという方針ですすめています。ViewやViewControllerのテストはデザインや仕様も頻繁に変わる現状だとコストが大きいのと、ViewとModelを分離する設計にしているので、Viewのテストは書かなくてよい、Model層やロジックはテスト書くようにしましょうといった方針にしています。
56. 右の図はiOSの週一定例のシートですが、急ぎでないものは各自思いついた時に議題を入れてもらって、定例で結論を出してアクションしていくといったやり方をしています。あとはGitHubやSlack上での議論から開発ルールを更新していっています。
57. 次にブランチ戦略についてお話します
58. シンプルにこのような形になります。masterブランチがあってQA期間中はqaブランチを使うといったやり方にしています。
59. これにsprintの図を当てはめるとこのような感じです 左が仮でスプリント2.2.0、真ん中がスプリント2.3.0、右が2.4.0になります
60. 真ん中のスプリント2.3.0を例に見てみますと
61. 2.3.0の開発期間中はmasterブランチからトピックブランチを切ってmsterにプルリクしてmasterにマージします
62. 2.3.0のQA期間中はqaブランチからトピックブランチを切ってプルリクエスト、qaブランチにマージします
63. 2.3.0QA期間中のmasterブランチは2.4.0（次のversion)の開発ブランチになります
64. 2.3.0の申請が終わったらqaブランチをmasterにマージします。 ここは申請というよりは適時マージしているイメージになります
65. このブランチ戦略の良いところとしては、プルリクエストのmergeができないということがあまり発生しないということと、基本はGitHubフローでシンプルでわかりやすい点です。スクラム開発のsprintサイクルにうまくはまるのではないかと思います。
66. えー、Beta配信ということで、テスト用アプリの配信の自動化とフローについてです
67. 図のような感じになります
68. まず　Developerがプルリクエストします。
69. レビューを通ってmergeされると、bitriseがhookでjobを起動します
70. 対象ブランチは先程お話したmasterとqaを対象にしていて、Crashlyticsにuploadして、TesterやDesigner、Directorなどがすぐに確認できるようなっています。 あとはmasterとqa以外にqa-xxxxといった名前でブランチを上げると配信するようにしていて、特別なversionで何か確認してもらいたい時などにも配信できるようにしています
71. あとはQA期間中はTestFlightにも配信して、本番と同じ状態で確認できるようにしています
72. はい、あとはSlackの通知もいくつか自動化しているものを紹介します
73. いくつかピックアップしたいと思います
74. 左上はCocoaPodsのライブラリのアップデートある場合に〜通知しているものです
75. これは最新のqaビルドが配信されたよ〜といったお知らせで
76. Crashがけっこう起きてるよ〜といったお知らせ、
77. これは自前のbotですが、レビューがしばらくされてないプルリクエストがあるよとお知らせしているものです。
78. あとはp12の証明書が切れそうだよ〜とか
79. App Storeのレビュー評価
80. iOS11のbeta10でたよ〜といったAppleのRSSなどをSlackに通知しています。
81. はい、最後にQAの効率化についての取り組みについてです
82. 機能が多くなり複雑になってくると、QAの時間も比例して大きくなってきてしまいます。その効率化にも取り組んでいて、デバッグメニューを容易しています 現状のデバッグメニューの機能についていくつかピックアップすると
83. その他としてカンファレンスへの登壇や、チーム内でのランチ勉強会などを行うことによって直接だけでなく間接的に知見の共有になったりしています
84. はい、まとめです。AbemaTVは事業的に先を見据えたサービスなので、その場しのぎで開発していると長期的な開発のどこかで破綻してしまう可能性もあると思います。なのでAbemaTVのモバイル開発においても長期的な視点で、開発スピードとアプリの安定性、コードの品質を保った継続的な開発ができるように意欲的に取り組んでいます。 簡単になりますがまとめということで、発表を終わりにしたいと思います