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

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

68b11f940176220aa84f755abb0bea10?s=47 Hiroki Otsuka
June 30, 2022
Programming
0
310

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

68b11f940176220aa84f755abb0bea10?s=128

Hiroki Otsuka

June 30, 2022
Tweet

Other Decks in Programming

See All in Programming
動画合成アーキテクチャを実装してみて
D5b3791309c8a03d37f05d238ded370d?s=48 satorunooshie
0
540
Introduction to Property-Based Testing @ COSCUP 2022
Fd9f21969cd8a0cbf0b2f48a9af0cc28?s=48 cybai
1
150
Pythonで鉄道指向プログラミング
Cc15f7fbb06e96bdb56992e3d42ea8cd?s=48 usabarashi
0
130
Enzyme から React Native Testing Library に移行した経緯 / 2022-07-20
41c9dedd1b4c53ace82e040bb09334fe?s=48 tamago3keran
1
160
これからのスクラムマスターのキャリアプランの話をしよう - スクラムマスターの前に広がる世界
93b2f8445b4096456c336ec2ea5dd0e7?s=48 psj59129
0
200
Git操作編
A3efba75cfbb0b7d5248eb74d1d29d79?s=48 smt7174
2
230
FargateとAthenaで作る、機械学習システム
2e7553bdade564572b26060b4a58b98e?s=48 nayuts
0
170
ファーストペンギンを志すものに伝えたい - １人目のアジャイル推進者がたどった成功と失敗
93b2f8445b4096456c336ec2ea5dd0e7?s=48 psj59129
0
100
ECサイトの脆弱性診断をいい感じにやりたい/OWASPKansaiNight_LT1_220727
341ac9f717b3904d6674784c0fcd9cf2?s=48 owaspkansai
0
290
Getting Started With Data Structures
Fbe54abf4094d19f1548783d072a7fa7?s=48 adoranwodo
1
260
SwiftUIで「意図」を伝える / swiftui_intention
42a6a049ac8f5265f31858a9509217fb?s=48 uhooi
2
140
RustのWebフレームワーク周りの概観
E42cffccdc14c60a449caac8de0ce0e0?s=48 hayao
0
170

Featured

See All Featured
What the flash - Photography Introduction
5ce91fa49a613cbc3e20d5f96856473f?s=48 edds
62
10k
How to Ace a Technical Interview
2f5463832ccb768ccb4a1ca3607c27ef?s=48 jacobian
266
21k
What's in a price? How to price your products and services
Dad095ea7038f89f760419ce475d5d14?s=48 michaelherold
229
9.4k
実際に使うSQLの書き方 徹底解説 / pgcon21j-tutorial
88f4e84b94fe07cddbd9e6479d689192?s=48 soudai
39
13k
Build The Right Thing And Hit Your Dates
016edace78ffe826f2348d1e0c504afd?s=48 maggiecrowley
19
1.2k
A Philosophy of Restraint
C9b18d9dff88a9dd2393364c2b3b21bd?s=48 colly
192
15k
For a Future-Friendly Web
F68cb25ae3e5c2106997454b13b7737d?s=48 brad_frost
166
7.5k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
B3d6434763caa0ef5dc4b792662c49f7?s=48 smashingmag
237
19k
Docker and Python
Ecdea9b9714877b86cee08458f085481?s=48 trallard
27
1.6k
The Success of Rails: Ensuring Growth for the Next 100 Years
C44e1f7e22c3f23cff7bc130871047ef?s=48 eileencodes
14
3.8k
Put a Button on it: Removing Barriers to Going Fast.
Bfd6b681ec6e8c9bef82ff0521c364f7?s=48 kastner
56
2.3k
Adopting Sorbet at Scale
2bb923c57a1fdc3a2eb484bb8d565fd2?s=48 ufuk
63
7.6k

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 • 解析のルールはレベルベース • レベルに応じて指摘の厳密さが変わる •

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