「速」を落とさないコードレビュー

Forkwell Meetup #3 https://forkwell.connpass.com/event/48147/Read less

[chinpokomon_master]

こんなのレビューでやろうとするからダメなんだよ。普通に話しにいけよ。

[masatomo-m]

これ、レビュアーやってる人はすごく同意できるだろうけど、PR出す側のみんながみんなこのレベルに達してるわけじゃないから難しいんだよね。自動検査で弾けるものは弾くとかはできるけど、その先は経験値も必要そう

[toshiwo]

これはホントいいまとめ。rubocopなんかは価値には直接寄与しないけどデフォであり、なければ緩やかに土台が崩れていく感じはある

[otihateten3510]

正直言うとコードレビュー文化が嫌い。設計レビューだけでいい（プルリク駆動はわかる）

[naopr]

自分がレビューする場合は、セキュリティ・パフォーマンスチェックはレビューの主目的に入ってる。むしろ仕様を満たしているかどうかをレビュイーに任せている

[escape_artist]

引用されていたらしい

[koogawa]

確かにWIPのおかげで手戻りが減ったことが何度もある

[nn0]

設計段階でwipのプルリクエストあると議論できるし、手戻り少なくていいですよね。

[masutaka26]

レビューで typo を見つけたら、指摘せずに commit して、直したよとコメントすることもある。こっちの方が早い。

[morishin127]

あわせて読みたい http://techlife.cookpad.com/entry/2016/08/17/111500

[kayamelo151515]

“ ”

[hibariya]

"自己効力感"

[kakutani]

粉雪の舞う大須演芸場の前に(なぜか)半袖Tシャツ姿で立っていた @onk さんが朗々と語りあげている設定で見返して新たな感動に震えている。カイゼン=カタっぽさがすごくいい。

[takatama]

指摘コメントの応酬で「コードレビューが終わらない問題」を具体的に改善していく話。ルーキーをコードレビューで育成するコツが素晴らしい。

[Schuld]

ドキュメントや契約書のレビューの参考に（なるかも）。

[ruicc]

最低限の品質、適切な型システム持った言語を用いて大まかな設計と問題の共有と型の適切な定義と型チェックと網羅性チェックとwarning消すのとlintあたりでやっていきましょう..

[joe-re]

良い話

[kabukawa]

品質という錦の御旗の元に、時間をかけてインデントやらスペースやらまで指摘をするのが正とする人は多い。でも、それって顧客に何を提供するのさっ？て考えてんのかなぁ、とも思う。

[june29]

めちゃくちゃいい…

[horimislime]

いい話

[toritori0318]

よかった

[efcl]

コードレビューのレイヤー化

[bufferings]

良さそう。あとでじっくり読む。

[nobeans]

良い

[ebo-c]

朝早いですね

[sylvan_l]

「速」を落とさないコードレビュー

[ya--mada]

長い、けどおもしろい

[eichisanden]

とても参考になった。うまく行ってるチームではここまで気を使ってないけど、チームの人間関係が悪かったり、価値を届けるよりも実装的な拘りすぎる人が多いとかチームなんかは対策が必要なんだろうな。

[endo_5501]

"常にベテラン→ルーキーの方向のレビューしか無い状態は健全ではない"

[ryuzee]

よくまとまってる良い資料だ！

[ryu39]

すごく参考になった。機械的なレビューはドンドン取り入れていきたい。

[kusamakura22]

「速」を落とさないコードレビュー

[nyomonyomo]

今年の流行は「速」なのかな

[wata88]

いいことしか書いてない(涙

[peroon]

自動化、ルーキー