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

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

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

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

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

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

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

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

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

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

2014/01/18 117 users レビュアー 両者 正体 違和感 職質アンチパターン

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