「気に入らないコード」をレビューする際にRed Hatのエンジニアはどうしているのか

「気に入らないコード」をレビューする際にRed Hatのエンジニアはどうしているのか：10個のヒントとは？ プロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合はどうしたらよいだろうか。Red Hatのソフトウェアエンジニアが、コードレビューを行うに当たって念頭に置くべき10のヒントを解説した。 Red Hatでソフトウェアエンジニアを務めるデビッド・ロイド氏は2019年7月8日（米国時間）、コードレビューを行うに当たって念頭に置くべき10のヒントを同社の開発者向け公式ブログで解説した。プロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合に役立つ指針だ。コントリビューター側としても参考になる。 これらのヒントは、客観的で的を射たレビューを行い、プロジェクトとその参加者を前進させるという観点からまとめられている。 （1）

[tumo300-500]

`「こんなコードを書いても、ソフトウェアエンジニアがあなた自身に合ったキャリアパスだと確信できますか」` 京都感ある

[endo_5501]

“悪意を表に出さない”

[teracy_junk]

コードレビューの話というより社会性の話ぽい

[tanishiking24]

こういうの大事だとわかってるけどとにかく疲弊するので一日に何個もOSSのPRレビューとかができない...

[hex]

あれ？悪い例に「Fワードで罵倒する」がない（やめれ / あ、(7)がマイルドにかかれたそれか

[masayoshinym]

悪い例がさすがに悪すぎる。

[onfi]

こんなことにエンジニアのリソース使いたくない

[ya--mada]

おっと、リーナスの悪口はそこまでだ

[deep_one]

コミュ力の話だった。一つ目は割と納得した。先に進むたびに「それコミュ力高いの？」と思い始めた。

[otherworld]

“悪い例：「こんなコードを書いても、ソフトウェアエンジニアがあなた自身に合ったキャリアパスだと確信できますか」”

[koogawa]

「こんなコードを書いても、ソフトウェアエンジニアがあなた自身に合ったキャリアパスだと確信できますか」こんなコードレビューは嫌だw

[netcraft3]

改善の余地があるコード問題だ