Upgrade to Pro — share decks privately, control downloads, hide ads and more …

エンジニアの心も繋ごう、グループアサイン【MIXI TECH CONFERENCE 2023】

エンジニアの心も繋ごう、グループアサイン【MIXI TECH CONFERENCE 2023】

MIXI TECH CONFERENCE 2023
にてお話したジ ワンジョンの資料です。

動画:https://youtu.be/83VWlNCtS2Q

https://techcon.mixi.co.jp/2023/d3-7

MIXI ENGINEERS

March 03, 2023
Tweet

More Decks by MIXI ENGINEERS

Other Decks in Technology

Transcript

  1. ©MIXI 良いコードレビュー • 技術的負債を最小限にできる • 設計などの技術情報が共有される • 不具合発生を事前に防ぐ • メンバーのスキルが向上する

    ◦ チーム全体の質が上がる ◦ サービス向上に向けて対応スピード・開発速度が上がる • 会社のコードとして意識が高くなる • コミュニケーションが円滑になる 5
  2. ©MIXI モンストサーバチームの既存のコードレビュー 8 働き方 基本リモート 人数 10~15名 開発言語 Ruby タイトル歴史

    2023年10周年を迎えます。 バージョン開発期間 (QA期間含め) 1〜1.5ヶ月 1案件の開発参加人数 1~2名 レビューの対象コード ツール、テストコードも含む マージ条件 最低限3名のApprove必須 コードレビュー 自由参加
  3. ©MIXI チームのレビューの課題 9 Pull Request 作成 コード レビュー レビュー 反映

    Pull Request マージ バージョン開発開始 コード締め切り みんな開発案件で忙しい
  4. ©MIXI リーダーの悩み 10 • 新しく配属した人をチームに、チームのコードになじませたい ◦ コードレビュー参加に自然と繋がって欲しい • コードレビューの参加率を高めたい •

    良いコードレビューで共に成長に繋がって欲しい • 締め切り直前にレビューコメントが集中するなどのレビューの課題を解決し たい もっと早く気づいていたら ... 締め切りなのにまだ残ってる! 今回全然レビューに参加してないけ どなんかあったか?
  5. ©MIXI 新人の悩み 11 • モンストサーバコードの難しいところ a. コンテンツの専門用語が多い b. DB負荷を避けるため色んな種類のキャッシュを使う c.

    ジョブキューを使う d. 分散DB構造 今のコードでいいの? 相談したいけど... 締め切りもうすぐなのにレビューのコメントが ...! レビュアーとして参加したいけど、自信がない
  6. ©MIXI グループアサイン実行期間と流れ 16 アイス ブレイク 開 発 振 り 返

    り 改 善 点 反 映 開 発 振 り 返 り 改 善 点 反 映 …
  7. ©MIXI 1回目で得られたもの 18 • メンバーのやる気 ◦ reviewのコメントが増加 ◦ 全員締め切り前に余裕持って終わった •

    気軽に相談できる雰囲気 • 難しさを感じる部分のハードル下げ ◦ 質問のコメントが増えた • メンバーの責任感 • メンバーのコミュニケーション力
  8. ©MIXI 1回目と違う状況 21 • アサインの量、難易度(増やしました) • お互いの案件チャンネルに入った ◦ 原則、担当者以外は案件チャンネルに入ってない ◦

    案件の結果だけではなく、コミュニケーション過程から結論まで共有で きるようにした • 口頭レビューを導入
  9. ©MIXI 2回目で得られたもの 24 • お互いのコード知ることで不在の時にお互い対応できた • 成果物だけではなく、開発過程も共有した ◦ 悩みポイント ◦

    設計の過程と方向性 ◦ デバッグ記録 • 口頭レビューの効果が確認できた ◦ 設計の整理 ◦ PRの意図の整理 ◦ デバッグ効果