タメになった
相手と自分の成長の意味で考えや癖も共有したいのでとにかく気になったところや理解したこともコメントするようにして、コメントであるとか直さなくて良いと書いてる。相手の認知負荷の問題は確かになあ。
半分くらい同意で半分くらいはうーーーん悩ましい
『問題の指摘をする、解決策は提示しない』『期待値調整と関係値構築で軽減できる』この部分難しい気がする
>コードレビューは所詮1対1のコミュニケーションのため、レバレッジもあまり効きません。丁寧すぎるコードレビューをするよりは、ドキュメントなどに時間を使うほうが効率的です。
解決策は掲示しないってのはあんまり同意できないなあ。特に複数の解決案がある場合。レビューってのは対話だし、直して欲しいとお願いするわけだから、どうして欲しいのかちゃんと言うのはマナーじゃない？
個人的には成長目的のレビューのラリーは反対派で、育成ならペアプロしたほうがいいと思ってる
“この話には前提として、「開発者の技術レベルが十分に高く、自律して技術的な問題解決ができる」ことがあります。幸いなことに、今いるメンバーは技術力が高く自走できる人が多いため” 解散！
モヤっとする内容だった。半分くらいはわかるが。
なかなか見かけない強い思想のレビュースタンスだ。部分的には参考になりつつも、自分はどうしても具体案まで提示したり、imo/nits も書いちゃうな。前者はともかく、後者はリンターやAIレビュー強化して無くしたい
同意できるところもあるけど、納期と成長の兼ね合いは難しい問題だと思った
ほぼ納得できる内容だった。補足にある前提についても
解決策を提示しない、がレビュアーの中にある正解を言うまで終われません、じゃないのであればいいかな。それを当てるゲームは不毛すぎるので。
とても良い。レビュワーの負担が減り、レビュイーの成長機会は増える。
視野が狭い
動作、仕様、アーキテクチャレベルでの指摘はさておき、コーディングルールにない書き方に関する細かい指摘は、Approveのブロッカーにしてほしくないかも。
時間対価値の高いコードレビュー - Hello Tech
タメになった
相手と自分の成長の意味で考えや癖も共有したいのでとにかく気になったところや理解したこともコメントするようにして、コメントであるとか直さなくて良いと書いてる。相手の認知負荷の問題は確かになあ。
半分くらい同意で半分くらいはうーーーん悩ましい
『問題の指摘をする、解決策は提示しない』『期待値調整と関係値構築で軽減できる』この部分難しい気がする
>コードレビューは所詮1対1のコミュニケーションのため、レバレッジもあまり効きません。丁寧すぎるコードレビューをするよりは、ドキュメントなどに時間を使うほうが効率的です。
解決策は掲示しないってのはあんまり同意できないなあ。特に複数の解決案がある場合。レビューってのは対話だし、直して欲しいとお願いするわけだから、どうして欲しいのかちゃんと言うのはマナーじゃない？
個人的には成長目的のレビューのラリーは反対派で、育成ならペアプロしたほうがいいと思ってる
“この話には前提として、「開発者の技術レベルが十分に高く、自律して技術的な問題解決ができる」ことがあります。幸いなことに、今いるメンバーは技術力が高く自走できる人が多いため” 解散！
モヤっとする内容だった。半分くらいはわかるが。
なかなか見かけない強い思想のレビュースタンスだ。部分的には参考になりつつも、自分はどうしても具体案まで提示したり、imo/nits も書いちゃうな。前者はともかく、後者はリンターやAIレビュー強化して無くしたい
同意できるところもあるけど、納期と成長の兼ね合いは難しい問題だと思った
ほぼ納得できる内容だった。補足にある前提についても
解決策を提示しない、がレビュアーの中にある正解を言うまで終われません、じゃないのであればいいかな。それを当てるゲームは不毛すぎるので。
とても良い。レビュワーの負担が減り、レビュイーの成長機会は増える。
視野が狭い
動作、仕様、アーキテクチャレベルでの指摘はさておき、コーディングルールにない書き方に関する細かい指摘は、Approveのブロッカーにしてほしくないかも。