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

大規模プロダクトにLinterを導入し運用している話

Avatar for Hiroki Otsuka Hiroki Otsuka
June 30, 2022
Programming
0
950

 大規模プロダクトにLinterを導入し運用している話

Avatar for Hiroki Otsuka

Hiroki Otsuka

June 30, 2022
Tweet

More Decks by Hiroki Otsuka

See All by Hiroki Otsuka
サービスと共にチームも成長する 〜New Relicを利用したサービスとチームの定量化〜
Avatar for Hiroki Otsuka hirokiotsuka
0
1.6k
PHP_LaravelのプロダクトでのNew_Relic活用事例
Avatar for Hiroki Otsuka hirokiotsuka
0
450

Other Decks in Programming

See All in Programming
ぬるぬる動かせ！ Riveでアニメーション実装🐾
Avatar for Kuno Ayana kno3a87
1
210
プロパティベーステストによるUIテスト: LLMによるプロパティ定義生成でエッジケースを捉える
Avatar for Tetta Noguchi tetta_pdnt
0
320
Compose Multiplatform × AI で作る、次世代アプリ開発支援ツールの設計と実装
Avatar for Takeshi Hagikura thagikura
0
150
GitHubとGitLabと AWS CodePipelineで CI/CDを組み比べてみた
Avatar for Satoshi Kaneyasu satoshi256kbyte
4
220
請來的 AI Agent 同事們在寫程式時，怎麼用 pytest 去除各種幻想與盲點
Avatar for Keith Yang keitheis
0
120
Azure SRE Agentで運用は楽になるのか？
Avatar for KAMEGAWA Kazushi kkamegawa
0
2.2k
複雑なドメインに挑む.pdf
Avatar for Yuki Sakai yukisakai1225
5
1.1k
Deep Dive into Kotlin Flow
Avatar for Matsuda Jumpei jmatsu
1
330
1から理解するWeb Push
Avatar for Minoru Takeuchi dora1998
7
1.9k
Introducing ReActionView: A new ActionView-compatible ERB Engine @ Rails World 2025, Amsterdam
Avatar for Marco Roth marcoroth
0
680
Navigation 2 を 3 に移行する（予定）ためにやったこと
Avatar for yokomii yokomii
0
160
速いWebフレームワークを作る
Avatar for Yusuke Wada yusukebe
5
1.7k

Featured

See All Featured
Six Lessons from altMBA
Avatar for Skipper Chong Warson skipperchong
28
4k
Speed Design
Avatar for Sergey Chernyshev sergeychernyshev
32
1.1k
"I'm Feeling Lucky" - Building Great Search Experiences for Today's Users (#IAC19)
Avatar for Dan Newman danielanewman
229
22k
It's Worth the Effort
Avatar for 3n 3n
187
28k
We Have a Design System, Now What?
Avatar for Morgane Peng morganepeng
53
7.8k
Navigating Team Friction
Avatar for Lara Hogan lara
189
15k
Scaling GitHub
Avatar for Zach Holman holman
463
140k
ピンチをチャンスに：未来をつくるプロダクトロードマップ #pmconf2020
Avatar for Aki / @LoveIdahoBurger aki_iinuma
126
53k
A better future with KSS
Avatar for Kyle Neath kneath
239
17k
Rails Girls Zürich Keynote
Avatar for Gregor Martynus gr2m
95
14k
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
Avatar for panda_program panda_program
112
20k
Why You Should Never Use an ORM
Avatar for John Nunemaker jnunemaker
PRO
59
9.5k

Transcript

  1. https://www.dip-net.co.jp/ 1 ⼤規模プロダクトにLinterを導⼊し 運⽤している話 ディップ株式会社 ⼤塚裕紀

  2. https://www.dip-net.co.jp/ 2 自己紹介 大塚裕紀（おおつかひろき） • 経歴 2018年4月からエンジニアとして働き出す 2020年9月にディップ株式会社に入社 ずっとPHPやってます

  3. https://www.dip-net.co.jp/ 3

  4. https://www.dip-net.co.jp/ 4 10年ほど運用

  5. https://www.dip-net.co.jp/ 5 数千ファイル

  6. https://www.dip-net.co.jp/ 6 EC2からFargateへ AWS Fargate Amazon EC2

  7. https://www.dip-net.co.jp/ 7 JavaからPHP&Laravelへ AWS Fargate Amazon EC2 + CMS

  8. https://www.dip-net.co.jp/ 8 ですが。。

  9. https://www.dip-net.co.jp/ 9 課題 • バグになりうるコードがある • 長期運用しておりコードが複雑 • テストコードが不十分

  10. https://www.dip-net.co.jp/ 10 課題 • バグになりうるコードがある • 長期運用しておりコードが複雑 • テストコードが不十分

  11. https://www.dip-net.co.jp/ 11 そもそもLinterってなに？

  12. https://www.dip-net.co.jp/ 12 Linterとは • 「lint」という静的解析ツールから派生し出来た言葉 • 静的解析を行うプログラムの総称 • コード上は問題ないけどバグになりうる部分を指摘し てくれるツール

  13. https://www.dip-net.co.jp/ 13 導入したLinter

  14. https://www.dip-net.co.jp/ 14 導入したLinter https://phpinsights.com/

  15. https://www.dip-net.co.jp/ 15 PHP Insights • PHP用の静的解析ツール • コードの品質やコーディングスタイルの解析に加え、 アーキテクチャや複雑さも解析してくれる •

    Laravel,Symfony,Yii,WordPress,Magento2などですぐに使 えるように設計されている

  16. https://www.dip-net.co.jp/ 16 なぜPHP Insights？

  17. https://www.dip-net.co.jp/ 17 なぜPHP Insights？

  18. https://www.dip-net.co.jp/ 18 なぜPHP Insights？ 実行結果が見やすい

  19. https://www.dip-net.co.jp/ 19 なぜPHP Insights？ • 細かくルールの設定が可能 • Code：コードの評価 • Complexity：複雑さの評価

    • Archtecture：アーキテクチャの評価 • Style：コーディングのスタイルの評価

  20. https://www.dip-net.co.jp/ 20 導入してどうだったか

  21. https://www.dip-net.co.jp/ 21 導入してどうだったか • 導入は容易に出来た • ルールを絞ることで無理なく導入出来た

  22. https://www.dip-net.co.jp/ 22 導入してどうだったか • 導入は容易に出来た • ルールを絞ることで無理なく導入出来た

  23. https://www.dip-net.co.jp/ 23 導入してどうだったか • 導入は容易に出来た • ルールを絞ることで無理なく導入出来た

  24. https://www.dip-net.co.jp/ 24 導入後の運用

  25. https://www.dip-net.co.jp/ 25 導入後の運用 AWS CodeBuild GitHub

  26. https://www.dip-net.co.jp/ 26 導入後の運用 テストと Linterを実行 PUSH HOOK GitHub AWS CodeBuild

  27. https://www.dip-net.co.jp/ 27 導入後の運用 テストと Linterを実行 PUSH HOOK GitHub AWS CodeBuild

  28. https://www.dip-net.co.jp/ 28 導入後の運用

  29. https://www.dip-net.co.jp/ 29 導入後の運用 コードレビューの負担が減る

  30. https://www.dip-net.co.jp/ 30 課題は？

  31. https://www.dip-net.co.jp/ 31 課題は？ • バグになりうるコードがある • 長期運用しておりコードが複雑 • テストコードが不十分

  32. https://www.dip-net.co.jp/ 32 課題は？ • バグになりうるコードがある ü バグになりうるコードはLinterで指摘される • 長期運用しておりコードが複雑 •

    テストコードが不十分

  33. https://www.dip-net.co.jp/ 33 課題は？ • バグになりうるコードがある ü バグになりうるコードはLinterで指摘される • 長期運用しておりコードが複雑 ü

    修正の土台が出来た • テストコードが不十分

  34. https://www.dip-net.co.jp/ 34 これからやっていきたいこと

  35. https://www.dip-net.co.jp/ 35 これからやっていきたいこと • ルールの厳密化 • 自動修正

  36. https://www.dip-net.co.jp/ 36 これからやっていきたいこと • ルールの厳密化 • 自動修正

  37. https://www.dip-net.co.jp/ 37 これからやっていきたいこと • ルールの厳密化 • 自動修正

  38. https://www.dip-net.co.jp/ 38 まとめ

  39. https://www.dip-net.co.jp/ 39 まとめ • コードに合わせて無理のない範囲で導入できた • コード全体の品質が向上した • コードレビューの負担が減った •

    自動化することでプロダクトの品質を維持できる • ルールの厳密化と自動修正もやって行きたい

  40. https://www.dip-net.co.jp/ 40 ありがとうございました。

  41. https://www.dip-net.co.jp/ 41 APENDIX

  42. https://www.dip-net.co.jp/ 42 他Linter • Larastan • 解析のルールはレベルベース • レベルに応じて指摘の厳密さが変わる •

    ルールを細かく設定できないのでパス