わんわおーん

Transcript

1. わんわおーん @Doarakko
2. ・2019年12月入社（2社目） ・エンジニア歴1.5年 　- 仕事: Go, PHP 　- 趣味: Python, C++ ・AtCoder 緑 ・海外サッカーをよく見ます 自己紹介 2
3. Q. 弊社リポジトリは 94 個あります（2022/5 現在）が CI に Linter が導入されているものは何個あるでしょうか 3
4. A. 2 個 4
5. 私たちは原始時代の開発をしています 5
6. 人がする仕事ではない in コードレビュー 6 ・インデントがズレてます ・import の並び順が違います ・この変数は使われてないので削除してください ・エラーハンドリング漏れてます ・etc
7. とは言いつつも... 7
8. 私はエディタの設定で Lint かけてます 8 ・全員がローカルで Lint の設定している前提での開発は現実的ではない ・実際問題できていない 🍧 ローカルでの設定と CI 両方必要です
9. Linter のカスタマイズ面倒くさい 9 ・面倒くさいならやらなくていいです ・まずはデフォルトのものを導入 ・運用しながら自分たちでカスタマイズしていきましょう
10. Lint されてないコードに導入するのしんどい 10 …
11. Reviewdog の出番です 11
12. Reviewdog とは 12 ・Linter の実行結果を PR のレビュー形式でコメントしてくれるツール ・各種 CI で使用可能 ・Linter の設定はローカルと同じものを参照します ・各種 Linter の GitHub Actions を用意してくれているので 　YAML コピペしてえいやで簡単に導入できる 🍎 https://github.com/reviewdog
13. 差分に限定して Lint をかけてくれる 13
14. 全ファイル修正しなくて OK 14
15. これだけです 15
16. 導入しましょう 16
17. こんなこと思ってませんか 17
18. うちのディレクトリ構成複雑だからめんどくさそう 18 ・YAML に実行するディレクトリを指定するだけです
19. 私しか触ってないリポジトリだからいいや 19 ・その成れの果てが今の弊社です ・あなたは一人ではありません、未来な仲間のために 🍧 Terraform や Ansible にも Linter はあります
20. 誰かがやるでしょ 20
21. あなたしかいません 21
22. わんわおーん @Doarakko