コードレビューにラベルを付けるだけでチームの心理的安全性を高めた話

ハコベルシステム開発部のおおいし (@bicstone) です。普段はフロントエンドエンジニアとして物流DX SaaSプロダクトの開発を行なっています。 この記事ではハコベルの開発チームが心理的安全性の向上を目的に採用した、プルリクエスト (マージリクエスト) コメントにラベルを付ける手法についてご紹介します。 背景 プルリクエストをレビューする時、レビュアーとして上から目線になってしまい相手を傷つけないか緊張したり、ちょっとした確認のつもりで書いたコメントが修正必須と捉えられてしまったりした経験はないでしょうか。 本来、ピアレビューは対等な関係であるはずなのに、レビューする側の方が上になってしまいお互いに恐縮してしまいがちです。「勘だと怪しいけど間違っていたら怖いから言えないな」や、「将来的に辛くなりそうな実装だけどわざわざ指摘するほどでもないな」など荒波を立てずにApproveしてしま

[sysjojo]

どういうつもりで言ってんの?がわかってとてもいいと思う。いちいち「個人的な意見なんでスルーしてもらっていいけど」と書くとめんどいし受け取る側でも邪推することあるけどタグならフィルタできるしね。

[zentarou]

いちいちラベルついててうぜーなと思ったことあるけど、単にその人のことが嫌いなだけだった

[shnwt]

優先度と期限を決めることは大事

[launcher]

小心者なので、この手ルールがあると助かる

[ymse]

返信テンプレート知らなかった 良さそう

[iww]

これこそ顔文字で表現出来たら一番良いんだけどな。 でも遠からず京言葉のようにラベルと本音が著しく乖離していきそう

[xlc]

私は古い人間なので、教育したいなら対面レビュー、非対面なら教育はあきらめ、承認/否認、否認なら修正が必要な個所を列挙、でよいと思ってます。

[gachapining]

「Saved replies（返信テンプレート）」個人設定単位じゃなくて .github でリポジトリ単位で管理できたら便利そうなのでなんか惜しい

[NOV1975]

対面でレビューして指摘入れるときはたいていそういうこといいながらやるし、書面でやるときも「これは単なる確認ですが」「興味本位ですが」みたいな書き方もするなあ。

[NetPenguin]

この手のわからなくもないけど「あの時私は言った」みたいな防御的な使われ方しないかの懸念があってあまり好きではない。レビューした人も同じくコードに責任を負うぐらいの感覚なら問題ないけど……

[tick2tack]

質問、参考、あら捜し、お手すきで、提案、必須、などのラベルをコメントに付けると。どう扱ってほしいか決めることでちょっとしたことでも言いやすくなるわけか。すごくいいね

[okbm]

返信用テンプレートとかあったのか

[asa_ca3]

なんとなく付けてたけど、明文化して書きやすくしようと思った

[cocoasynn]

自分もこの手のラベル付けしてる。おおむね同じだけど良かった点を褒めるときとか感想をただ述べたいとき用に[good]とか[感想]とか使ってる

[saikyo_tongaricorn]

京言葉化するとかこわいこと書くなよ！

[kondoly]

どれくらいの規模でどの程度の練度を目標に教育や訓練の結果運用に至れたのか気になる。

[pixmap]

こういうちょっとしたルールでできる工夫が素晴らしい。すべてのコメントに対して、このルールに従ってラベルをつけることが強制でない、というのも重要な点。

[SundayIsEveryday]

構成管理はSVN、文書はwordやexcelのレガシーな現場ばかりで働いてきたけど、どこもレビュー議事録にラベルの選択欄があった。開発プロセスがしっかりしてる業界だからか、レガシーでもこの辺は成熟してたのかも。

[Cald]

OGR(大喜利)は？

[youichirou]

いいね。「この分野は素人なのですが」とか良くない。

[atsushieno]

ローカル文化が積み上がると新規参入しづらくなるので、グローバルに通用するラベルをRDFで適用できるようにセマンティクスを規定したやつがほしい（いらない）

[juejue]

ラベル付けって当たり前かと思ってたけど、当たり前じゃないんだな...

[cider_kondo]

真面目な良い記事だと思うのだがなぜか　MSG:門外漢の素朴な疑問　SK:素人質問で恐縮ですが　みたいなのを連想した。要は、チームとしてのまとまりがある場合とそうでない場合で変わる、ってことだよな、つまり

[zyzy]

切り分けた事というか、ラベルの切り分け方が良い案件

[natu3kan]

優先度は低いけど万一があったらイヤだから言っておきたいみたいなのあるかもだもんね。

[DiceK_U]

ごちゃごちゃするだけで形骸化一直線

[mr_mayama]

IMO: いいね。ラベル3個ぐらいにならんか

[sds-page]

普段からチームメンバーで雑談の場を設けるなどした方が心理的安全性は高まるかと思います

[t_motooka]

大きめの組織だと、こういう慣習はあった方が良いよね〜

[Shinwiki]

“q: ここって、こうで良いんでしたっけ？追えきれておらず...” よくなかったです！すぐ直します！！(チッ、イヤミくせえ指摘しやがって)。 こうなります。

[mas-higa]

Q: Saved replies ってどうやって設定するんでしたっけ?

[masadream]

コメントの趣旨をラベル付け。よい工夫！レビューする方も受ける方もストレスがかなり減りそう。

[remonoil]

"GitHub では、コメントのテンプレートを設定して、キーボードショートカットやツールバーから呼び出すことができる Saved replies という機能が存在します。"

[tockri]

いい習慣だなー。

[zkq]

これいいな。小さいアイディアと負担で大きな効果。

[showgotch]

天才か？会議に使える👏マストとベストエフォートとウィル、位の仕分けは考えたことあるけどこっちの方が使えそう

[raamen07]

直せと言っているのか呟いているだけなのか何して欲しいのかよくわからんコメントがついてapprove無しだと困るんだよね。approveしてないんだから察して直せよみたいな事してると不満も溜まっていくし

[bascinet]

TextBlazeで/mustと打ったらshields.ioのバッジでmust出すようにしてるなぁ