コードレビュー - hitode909の日記

コードレビュー，慣れるとできるけど，いきなりdiffを渡されて，どうぞ見てくださいと言われてもよくわからないと思う． やりましょうというのはいいけど，ただむやみに読んでもうまくいかない．変更がある程度大きくなるとdiffだけ見てもよくわからないので，いろいろ見ることになる． 僕はいつも以下のようなことを無意識にやってて，うまくいってる気がしてる．GitHubのPull Requestの仕組みを使ってる前提で． Discussionをさらっと眺めてどういう問題を解決したいのか見る Commit Statusを見て，テスト通ってることを確認する Commitsタブで1コミットずつブラウザの新しいタブに開く 全部クリックし終わったら古い順に1コミットずつ読む 気になる点があったらエディタとかにメモしておく．あとで書き直されるかもしれないので，まだコメントしない 全コミット見終わったらFiles

[hitode909]

コードレビューの記事に正しい意見ついた

[takeda25]

雰囲気のいい職場にいる者は幸いである

[kamipo]

これやってる「1コミットずつブラウザの新しいタブに開く」「古い順に1コミットずつ読む」

[papilio17]

“納品して完成みたいなものではなくて，毎日リリースしてるくらいで，ちょっとずつ変わっていくもの”

[kazuhooku]

inspiringでいい記事

[sanemat]

コミット別タブ開きはやろう

[skit_n]

コミットの歴史が多いとどう読んだらいいだろうとモヤモヤしてたけど、この記事読んでスッキリした。

[leokun0210]

すごい

[non_117]

物理コミュニケーションも大事にしたい

[Windymelt]

よむ

[BigFatCat]

"下巻の最初のほうに，まともなコーディング規約とは，とかそういうことも書いてあって，その章読むだけでも役立つ．" オブジェクト指向入門 ポチっちまった。

[elim]

よさある

[haishi]

コードレビュー，慣れるとできるけど，いきなりdiffを渡されて，どうぞ見てくださいと言われてもよくわからないと思う． やりましょうというのはいいけど，ただむやみに読んでもうまくいかない．変更がある程度大きく

[urouro_n]

GitHub / "コミットを順番に見ていくと書いた人と同じ気持ちになれるので良い” "動くならまずマージしてしまって，そこから落ち着いてリファクタリングのissueを作ることが多い"

[efcl]

githubでのコードレビュー手順 :shipit: diff,コミットの読み方

[kasumani]

コードレビュー コードレビュー，慣れるとできるけど，いきなりdiffを渡されて，どうぞ見てくださいと言われてもよくわからないと思う． やりましょうというのはいいけど，ただむやみに読んでもうまくいかない．変更が

[yatemmma]

コメントにミサワを張りたい衝動

[ryonext]

コードレビューのいいやり方気になってたので超有益情報ですわー

[koogawa]

「コミットを順番に見ていくと書いた人と同じ気持ちになれるので良い」やはり意図が伝わるコミットって大事よね。オレも最高のコミットを目指したい

[hush_puppy]

"読みにくいコードとかを書く人はそもそもいないので" 羨ましい妬ましい。

[murasaki11]

とても参考になる

[inamem9999]

関係ないところでFirefoxからChromeに切り替えようかと思ってしまった

[Nyoho]

Chrome 拡張が気になって次が読めなくなってしまった! また戻ってきます!(←どうでもいいだろっ)

[t-wada]

Github の PR を実際にレビューするやりかた。実践的でとてもいい(Commits別タブ開き私もやる)。参考文献に定番中の定番『リーダブルコード』だけじゃなく『オブジェクト指向入門第2版』があるのが渋い

[side_tana]

参考になる

[uasi]

ひとでくんさんがどうやってコードレビューしてるかの詳細、よい

[enk_enk]

ライブ感あってよい

[htomine]

“リーダブルコード ―より良いコードを書くためのシンプルで実践的なテクニック (Theory in practice)”

[vivit_jc]

コードレビューの仕方。参考になる。

[UDONCHAN]

ファンタスティック

[hkataoka]

いい記事