これは、asumikam #phpcon_odawara Advent Calendarの23日目の記事です🎄 本日は「最近コードレビューで指摘されたことをシェアします」の感想を書いていきます!
感想
- PRのファイル数減らして
- PRの概要書いて
- テストコード書いて
- 未使用のuse消して
- PHPDoc追加して
- このログは不要では?
- 最新バージョンの書き方に揃えて
- N+1起きない?
などなど・・・、さまざまなコードレビューの"あるある"指摘は、自分自身へのコードレビューで防げる、という話でした。 本質的じゃない指摘を減らして、コードが実際に合っているか、のところにレビュアーの時間を割きたいですよね・・・! 上記にあげてくれた指摘は、PHPStanやphpcsたちも、無くす方向に加担してくれそうです💪
yukaさんの言葉で好きだったのは「自分自身でレビューすることで自分の傾向がわかる」というものです。 ある程度計測してみるとどんな時に自分がミスしやすいのか、がわかって良さそうですね〜〜。
それにしても、はじめてのLT登壇とは思えない落ち着きでとても聞きやすかったです。 会場の雰囲気作りも込みで、"喋り"のうまさをかんじる登壇でした。
kadokuraさん、登壇ありがとうございました
当日は若干接続トラブルありつつでしたが、会場のみなさんのヘルプもあって登壇成功!!でした!!! 会場のあったかみをかんじました、ありがとうございました🙇♀️✨
そして〜〜〜ぺちおだ2025でも!!yukaさんのLT登壇がきまっております👏 めちゃくちゃ楽しみです、2025年もよろしくお願いします!!!