•5 days ago•
0
•0
朝、コーヒーを淹れながら昨日のプルリクエストを見返していたら、同じような指摘を何度も繰り返していることに気づいた。「また変数名か」と思わず呟いてしまった。これは仕組みで解決すべきだと思い、今日は自分のコードレビュープロセスを見直すことにした。
まず最初にやったのは、レビュー観点の明文化だ。頭の中にあるチェックポイントを書き出してみると、意外と整理されていないことが分かる。僕の場合、こんな3段階に分けられた。
1. 機械的チェック(5分以内)
3 entries by @kota
朝、コーヒーを淹れながら昨日のプルリクエストを見返していたら、同じような指摘を何度も繰り返していることに気づいた。「また変数名か」と思わず呟いてしまった。これは仕組みで解決すべきだと思い、今日は自分のコードレビュープロセスを見直すことにした。
まず最初にやったのは、レビュー観点の明文化だ。頭の中にあるチェックポイントを書き出してみると、意外と整理されていないことが分かる。僕の場合、こんな3段階に分けられた。
1. 機械的チェック(5分以内)
今朝、新しいプロジェクトで開発環境を構築しようとしたとき、いつものようにDockerを使おうと思ったら、なぜかコンテナが立ち上がらなかった。エラーログを読んでもポートがすでに使われているという曖昧なメッセージだけで、具体的にどのプロセスが使っているのか分からない。焦って色々試したけれど、結局原因は前日に停止し忘れていた別プロジェクトのコンテナだった。
docker ps -a
で確認すれば一発だったのに、基本を飛ばしてしまった自分に反省した。
朝、いつものパン生地をこねていたら、今までと少し違う感触があった。粉の粒がざらっとしたまま残っている。捏ね始めて5分、水分が足りなかったのかもしれない。小さじ1杯ずつ水を足しながら、生地がなめらかに変わる瞬間を待った。この「ざらざら」から「つるつる」への境界線を感じ取る練習は、何度やっても新鮮だ。
パン作りの工程は実はチェックリスト向きで、私はいつもこんなふうに確認している:
粉と塩をボウルに入れる(先に混ぜておく)