コードレビューで心がけている3つのこと【PHPカンファレンス協賛記念ブログ！】 - コネヒト開発者ブログ

こんにちは！エンジニアの @fortkle です。 あの伝説のゲーム「メダロット」のスマホゲームのリリース日がついに 2020年1月23日と発表がありました!*1 いまからワクワクしてきましたね！リリースしたらぜひロボトルしましょう！ さて、今回の記事は「コードレビュー」についてです。コネヒトに入社してから早4年、数百のPRをレビューしてきてだんだんと自分の中でコードレビューに対する接し方が定まってきました。今日は私がコードレビューで心がけていることについてご紹介できればと思います。 レビュワーとして ① "既存コード" の 歴史的経緯を素早く紐解く コードレビューは様々な目的で行われますが、「欠陥・バグを検出すること」「コードの改善」に期待をしていることが多いかと思います。 これらの目的を達成するためには、既存・変更後のコードの実装意図や背景を理解することがとても重要になります。特に長年

[fortkle]

書いたよ！

[vanbraam]

いい話.本当にwhy大事; 動かすのも大事.ローカルだけで動くものなら動かすし,動かすのが難しいものでもテストは走らせる

[yimajo]

nitsとか書かれても何の意味かググってもわかりづらいし、それが伝わるという前提あるならいいんだろうけど、ないんだったら「再レビュー不要です」って書けばいいと思う。予測変換ですぐだし

[plane25]

“レビュイーとして心がけていることは、今週末の2019年12月1日(日)に開催されるPHPカンファレンス東京 2019でLTとして発表します！” ですってよ！！！ ”