コードレビューガイドライン #loupestudy

株式会社LOUPEの社内勉強会です。 http://lo-upe.hatenablog.com/entry/loupestudy-codereview (参考) 眼鏡なしのコードレビュー http://postd.cc/code-review-without-your-glasses/ …

[deep_one]

その程度で「メソッド長すぎ」なの？

[mktakuyax]

見返している

[motchang]

眼鏡なしのコードレビュー良いですね

[msykt]

"眼鏡なしのコードレビュー"知らなかった。読んでみよう

[kamip]

いい話

[koheisg]

みんな同じようなことを言ってるけど、学びがある。

[decoy2004]

『技術的負債を残さないためのチームワーク』

[two-pack]

「眼鏡なしのコードレビュー」がいい。 / コードレビューガイドライン

[lanius]

複雑さ、パフォーマンス、セキュリティ。

[otchy210]

ルールちょっと厳しすぎる気もしたけど、眼鏡無しの雰囲気が Ruby なのでそんなもんかな、とも思う。言語依存。Java だともう少し緩めたいな。

[kozy4324]

いい話

[kiyo_hiko]

"Sandi Metzルール"

[lo-upe]

#loupestudy

[luccafort]

きちんと例外のルールも定めているの好感を持てる。例外ってどうやっても発生するわけでそこの部分がケア出来ずに原理原則通りやってると精神的によくない。

[h5y1m141]

コードレビューすることが出てきたからこういう考えが知れるのはすごい参考になる。スライド最後の「自然にコードレビューをする文化を作っていきましょう」が特に印象に残る

[yamitzky]

すごい、正論だ、、、書き方へのレビューは「どっちでもいいし時間の無駄」WIPは「間違えてマージするかも知れないから禁止」みたいなこと言われてるから共有しとこう

[kentana20]

ステキ

[kasumani]

コードレビューガイドライン #loupestudy // Speaker Deck Other Presentations in Programming Featured Presentations Tags: from Pocket October 16, 2014 at 11:00AM via IFTTT