下から目線のコードレビュー - steps to phantasien

WEB+DB の新しいやつがちょっと前にでてます. コードレビュー特集だそうな. 時が経つのは早い. まだ次の原稿書いてないのに… そういえば前にコードレビューの話を書いた気がして, 見なおしたところ かきかけ だった. せっかくなので続きを書いてみることにします. といっても何書くつもりだったか覚えてないのでだらだらと. WEB+DB PRESS の特集は, 主にこれからコードレビューを導入したい人に向けて書かれている. 幸か不幸か私はコードレビューを義務付けれたプロジェクトで働いているため, 導入には苦労していない. かわりにレビューをちょろまかせない面倒はある. ある意味でコードレビューを <やらされている>. もちろんこの言い分は大げさだ. 必要性に異議を唱える気はない. ただ異議はさておき自分の意向とは無関係にコードレビューに参加している気分を書いた話は あまり目にしないので,

[kiyo_hiko]

「コミットログは嘆願書だと思って書け」

[zorio]

コミット前にパッチ単位で見る方法だと、パッチを作成する過程でSCMの恩恵に預かれないので、githubのpull requestみたいなのが羨ましくなる。

[bowbow99]

ヒトリストなのでレビューしたりされたりすることは無いけど参考になった; タイトルから新人や半人前にレビューさせるのは教育によろしいんじゃないかと妄想した。

[waniji]

とても良い記事

[devorgachem]

素晴らしい記事。一つだけ思ったのが、「主観的にパッとしない」の部分はチーム間での情報共有を目指したものじゃないですかね。強制的にコードを読む機会を設けるのは悪いことではないと思う。

[haya14busa]

めっちゃよい

[oinume]

ためになった

[saku_na63]

わかりがある / "そして指摘が少なく, いつもすぐ LGTM をくれるレビュアは, ありがたいと同時に少し不安になる."

[key_amb]

この辺の言葉づかいにまだ慣れてないところあるなぁ。

[progrhyme]

この辺の言葉づかいにまだ慣れてないところあるなぁ。

[nilab]

下から目線のコードレビュー - steps to phantasien

[an_old_pencil]

うんうん

[xxxxxeeeee]

コードレビューについてのいろいろ

[invent]

下から目線のコードレビュー - steps to phantasien

[mumumu-tan]

[system:unfiled]

[moriyoshi]

いまさら読んだがきわめて良記事であった。

[arikui]

[]

[Rockridge]

「自明な変更に対しては, レビューを頼む側だけでなくレビュア自身も “ハンコおしときました” と言うことがある. Mozilla 系列の文化では LGTM のことを r=me という.」

[uasi]

コードレビューのイディオム

[fukaoi]

コードレビュー処世術、ようやく読んだ。”がっつり書き換えたいコードはまず別クラスに切り出せ!”はよくやる。

[t_f_m]

"レビュアに “ここリファクタリングした方がよくない?” と言わせる方法. 指摘をうけてリファクタリングしたパッチは大きくなる. でもリファクタリングを言い出したのはレビュアだから問題ない. レビュアをハメろ! "

[ogijun]

うう、天上の議論だ。がんばる...

[honeybe]

途中だけ読んだが良記事っぽいので後で読む。

[peroon]

”コミットは意味のある範囲でなるべく小さくする”

[udzura]

なにやらぼくの記事の未熟な点ををフォロー(post-commit review)していただいたような感想を受けました。取り上げていただきありがとうございました！

[tak4hir0]

via stomitaさんのアンテナ — Myはてな http://h.hatena.ne.jp/shinichitomita/following?location=http%3A%2F%2Fwww.hatena.ne.jp%2Fmy%2F

[naka-06_18]

LGTM with Nits はこの記事で初めて知ったのよ。

[TsutomuOnoda]

下から目線のコードレビュー - steps to phantasien

[Layzie]

ステキすぎる

[kei-s]

やっぱりおもしろかった

[raitu]

俺は下っ端レビュアだからエディトリアルかあからさまな設計書との矛盾しか突いてない。上司はガッツリ構造ごと否定する指摘書いてくる。役割分担気味

[t-wada]

ソフトウェア・エッセイスト(と私が勝手に呼んでいる) omo さんの新作。コードレビューに関するエッセイでもあり、かつパターン/イディオム集のような体裁でもある素敵な文章。