チーム開発のコードレビューにおける些末なコードレビューを避けるための提案

59cbc6c10d2d1ab75aeaf6c4b7515490?s=47 d9magai
June 26, 2019
120

 チーム開発のコードレビューにおける些末なコードレビューを避けるための提案

59cbc6c10d2d1ab75aeaf6c4b7515490?s=128

d9magai

June 26, 2019
Tweet

Transcript

  1. 1 チーム開発のコードレビューにおける些末な コードレビューを避けるための提案 と最後に会社紹介 令和元年6月 熊谷大地@d9magai 〜多様な開発メンバーの 良いものを作りたいという思い〜

  2. 2 チームでの開発 • 利点 ◦ 規模の大きな開発が可能に ◦ 多様なメンバーによる多様なスキル、多様な意見 • 課題

    ◦ コミュニケーションに起因する問題 ▪ 情報の共有、進捗・スケジュール管理の困難さ ◦ コードに関わる問題 ▪ コードの競合・差異を統合して解消する ▪ スピードを保ちつつ品質を維持する
  3. 3 チームでの開発 • 利点 ◦ 規模の大きな開発が可能に ◦ 多様なメンバーによる多様なスキル、多様な意見 • 課題

    ◦ コミュニケーションに起因する問題 ▪ 情報の共有、進捗・スケジュール管理の困難さ ◦ コードに関わる問題 ▪ コードの競合・差異を統合して解消する ▪ スピードを保ちつつ品質を維持する ←頑張れ
 ←GitHub使え
 ←今日する話

  4. 4

  5. 5

  6. 6 コードレビュー 書いたんで 見てちょ おけまる! • コードを誰かに見てもらう ◦ GitHubが多そう •

    自分の書いたコードを誰かが見る ◦ 見られることを意識 • 誰かが書いたコードを自分が見る ◦ (不完全ながら)コードを共有
  7. 7 コードレビューに関する議論 • どういった観点でレビューするといいか? • 誰が誰のコードをレビューするといいか? • どれくらいの時間をかけると良いのか? • 指導、教育、品質の担保をレビューで賄えるのか?

    • そもそもコストに対して見合うメリットはあるのか?
  8. 8 コードレビューに関する議論 • どういった観点でレビューするといいか? • 誰が誰のコードをレビューするといいか? • どれくらいの時間をかけると良いのか? • 指導、教育、品質の担保をレビューで賄えるのか?

    • そもそもコストに対して見合うメリットはあるのか? 色々あるけど、共通しているのは やるなら些末なコードレビューはしないようにしよう というところ
  9. 9 些末なコードレビュー • 半角スペースが必要なところにない • 改行するべきなのにしてない • インデントの字下げが間違ってる これらの指摘が無駄だとまでは言わないが・・・

  10. 10 些末なコードレビュー • 半角スペースが必要なところにない • 改行するべきなのにしてない • インデントの字下げが間違ってる これらの指摘が無駄だとまでは言わないが・・・ •

    この設計は拡張に対して開いていない • このコードはパフォーマンスに悪影響が出る • このテストは第三者が見たときに要求が分かり難い こういったレビューを目指したい
  11. 11 些末なコードレビューを避けるために 些末なコードレビューは機械的にチェック可能→自動化! pushする circieciで構文チェック 結果をコメント 些末なコードレビューを自動化して、本質的なレビューに集中

  12. 12 コードレビューを取り扱うサービスいろいろ https://sider.review/ https://scrutinizer-ci.com/ https://www.codacy.com/ • あくまでも自分で構文チェックしたい場合、 Pythonなら、flake8-checkstyle がオススス •

    私が作りました←これが言いたいだけの LTだった
  13. 13 自己紹介と会社紹介 • 名前:熊谷大地 ◦ 宮城出身 ◦ 一児の父 • 所属:  千株式会社

    ◦ ものづくり部 幼稚園・保育園向け写真販売サービス
          を運営してます
 ご静聴ありがとうございました エンジニア募集中です