レビューの仕方

Open8 勉強会で発表したレビューの仕方と心理的安全性の話しです。

[a-kuma3]

んー、とても共感するのだけれども、スキルかモチベーションのどちらかが同じ方向に向いてないと成り立たない、と思ってる。砂漠に水を撒くようなレビューですり減ってる人はいっぱい居るはず

[tameruhakida]

コードじゃないけど、トップコメの「砂漠に水を撒くようなレビュー」に泣いた。レビュアーの手間とか一切考えず、投げたら終わりと思っている人（当然質も低い）のレビューは苦行でしかない

[tpircs]

これに異論を唱える人はいないと思うんだけど、そうだよねそうだよねって読んだ後に残る物もあまりなかったりする。厄介な人をどう中和したとか、厄介だった自分がどう変われたか、みたいな話を読みたいな。

[kamiokando]

発送がとても早くて助かりました。まだ使ってませんが期待をこめて星五つ。

[odakaho]

前職に怠惰で攻撃的なレビュアー(PRは何日も放置、コメントは「こんなことも知らないの？」など攻撃的)がいてチームがぶっ壊れたのでニックの排除は極めて重要。対策は2階層上の上司にチーム替え提案するとか。

[kamezo]

はてなだからどうせ「ショップのレビューと商品のレビューを区別しろ」とかいう話とは違うんだろうなと思いながら開いた。やっぱり違った。あれはどうすればいいんだ。プラットフォーム運営が悪いんだろうけどさ。

[Bryntsalov]

意見が対立したら取り敢えず折れるようにしている。問題が発生した後に、ほれ見たことかと修正PRを出す。そうしないとわからん人は一定数いるし、自分がその状態の時もある。

[kaz_shu]

レビューコメントに 🤔 を使われるとイライラする。

[yosuke_furukawa]

Open8勉強会で発表したレビューの仕方の話です。

[ihmaiwtd]

凄くいい。気持ちは持ってるけど、言語化するのを忘れてた。コードに限らず、標準化されるような全てのレビュー成果物に当てはまると思っている。

[respectbookmark]

とても実践的。

[Kesin]

“とりあえず最初の返事を返そう” これは嬉しい。既読マークの機能がないからレビュー依頼を認識してくれているのかどうかを毎回Slackとかで聞いてる

[naokton]

レビューするとき細かすぎる傾向があるので、一歩引いて考える時の参考になる

[minase_mira]

IMOみたいなのをとりあえず書きがちだから気をつけないと

[taketack]

レビュー

[proverb]

自分のコメントでメンバーを不快にさせたくない、でも色々指摘しなきゃいけない…って考えて🙇‍♂️をやたら使う癖が付いてしまった。これはこれでウザいと思われてそうだけど。

[nilab]

「Open8 勉強会で発表したレビューの仕方と心理的安全性の話しです」

[ryan_aircloset]

これはめっちゃ同意。1日以内で返信できてないこともしばしばあるので自分を戒めるためにも。

[FunnyBunnyDizzy]

プロジェクトが始まったら最初の1ヶ月だけみんなでモブプロをやると、言われそうなことや価値観がだいたい予測できるようになってレビューの同期コストが大幅に減るという研究が（ありません）

[umai_bow]

そのコードベースがマージされて自分が（プロダクトが）耐えられるかという基準でレビューすればいいと思ってる。あとは教育

[shikiarai]

チームビルディングなんぞできたことがない

[tonaxi]

助かる。色々なことに横滑りできそう。

[omron]

モヤッとしたところが言語化されてて良き

[fujihiro0]

概ねいい内容。心理的安全性のために排除される怠惰や愚痴を言う人だれが決定するのだろうか。「あるべき姿を追い求めつつ」とはベストを追い求めているのではないのか。

[baccho07]

デザインレビューも同じ事が言える。あっちは感覚で話されると議論にならないので言語化するし、言語化を促しながら意見の真意を読み取る

[turanukimaru]

新人にレビューさせたら「Logger.debug(略)は仕様書に無いので消せ」というコメントが大量について超イラっ☆とした。素人の場合は心得だけじゃなくて具体的なやり方も教育しないとレビューは時間の無駄ぢゃねぇかな。

[raitu]

コードレビューの基礎的な話として読めた。かなり基礎的だからコードだけでなくレビュー全般にも適用できる。特にチームビルドにつなげる話など

[roshi]

そもそもレビューという行為自体が人との摩擦を生むだけで、かけるコストに対して品質や生産性に貢献しているのか疑問なんだよなー、とか言うと排除対象になるんだろうけど。

[tumo300-500]

やや横だけど `チーム全員が話し、話す量も同じ` これ大事。数人でほとんど喋ってしまう状態を心理的安全性が高いと思ってしまいがち

[fikah]

ニックをどうやって排除するかが知りたい。ニックみたいなタイプに限ってなぜか上からの評判は良かったりするから辛い

[dexia2]

フロントエンドなので、ユーザーの観点、テスターの観点、エンジニアの観点の三回でレビューするようにしてます。レビューする前には必ずテスト設計をする癖がついているので、スライドとは観点が違うかもしれない

[dpdp]

批評はreview、宝塚のはrevue

[y-mat2006]

自分が知らず知らず、そうなっていないか？＞「攻撃的な態度を取る役」「怠惰な役」「何かと愚痴を言う役」の３つをそれぞれの会議で行った。

[mohno]

わりと普通の話だった。

[yasu-log]

愚痴を言う人は周りの生産性を下げる。

[koKekkoh]

聞いた頃のある内容の復習。心理的安全氏を担保するマネージャの心理的安全性について、と天の邪鬼なことを考える。

[tmayuko312]

中和活動を自らずっとやってたんだけど、元上長のせいで私が病んだ時に、中和活動する人が居なくなって、元上長が病んでいったのを感じた時は「ふ～ざまみろ〜！」って思っちゃった思い出