コードレビュー時のコメントの意図を明確にする

2023/02/15 29 users 意図 疑念 コードレビュー時 コメント プレ

コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか？」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物... 続きを読む

コードレビューとPull Request、そしてその承認機能の副作用について考える - 時計を壊せ

2021/11/07 161 users pull request tl;dr レビュアー 時計 副作用

用語 レビュアー 対象となるコードをレビューする人のことを指します。 レビュイー レビューを受ける人、つまりレビューする対象のコードを書いた人のことを指します。 tl;dr アプリケーション開発業務におけるコードレビューはコードの正しさや質そして一貫性を保ち、それらと同時にコードに対するチームとしての共有知... 続きを読む

コードレビューは「コードの欠点を指摘する行為」ではない - id:anatooのブログ

2019/01/16 297 users anatoo レビュワー 欠点 実力 行為

コードレビューを「コードの欠点を指摘する行為」だと無意識に思っている人を見かけるけども、そういうふうに認識しないほうがチームにとって良いですよ、という話。理由は以下。 レビュワーの方がレビュイーよりも実力が無いといけない、という認識と結びつきがち チームの若いメンバーがレビュワーになりづらくなる 古... 続きを読む

レビュー文化の会社で、楽に生きられる思考法 - ビープラウド社長のブログ

2018/03/19 43 users ビープラウド社長 レビュアー 思考法 人格攻撃 マシン

2018 - 03 - 19 レビュー文化の会社で、楽に生きられる思考法 仕事論 「コードを憎んで人を憎まず」 コードレビュー時のレビュアー（レビューする側）の考え方（レビュー時に人格攻撃にならない、感じられないように気をつけるなど）として、良く使われる言葉です。 一方で、レビュイー（レビューを受ける側）はどのように考えればよいでしょうか。 マシンやプログラムからエラーメッセージが出ることを 「怒... 続きを読む

コードレビューにおけるレビュアー側のアンチパターン – ITアーキテクトブログ – Medium

2018/01/03 645 users medium GitHub tl;dr レビュアー 例外

tl;dr コードレビューが上手く回って無くてチームが疲弊して辛かったよ レビュアーの言い方を変えるだけで大体解決するよ 立場とかで例外を許さず、みんながレビューしてレビューされると良いよ はじめに あるプロジェクトでGitHubのPRベースでのコードレビューを導入をしました。いかんせんチーム開発が初めてレベルの新人さんが多く、何かと苦労しました。特にレビュイーに対して不効率な指摘はそのまま指示の... 続きを読む

レビュアーとレビュイーの関係に関して - 職質アンチパターン

2014/01/18 117 users レビュアー 両者 正体 違和感 立場

2014-01-18 レビュアーとレビュイーの関係に関して 感じていた違和感の正体がわかった． レビューにおいて，レビュアーとレビュイーの関係には上も下もない． レビューという場では，両者の立場は対等でなければならない．さもなければ，「このレビューおかしい気がするけれど，あの人は立場が上だから指摘しにくい」だとか，「相手は格下だから適当にレビューしても良い」だとか，そういう良くない雰囲気が形成され... 続きを読む