気に入った記事をブックマーク
- 気に入った記事を保存できます
保存した記事の一覧は、はてなブックマークで確認・編集ができます
- 記事を読んだ感想やメモを書き残せます
- 非公開でブックマークすることもできます
エントリーの編集
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント87件
- 注目コメント
- 新着コメント
hiroomi
"リーダーや他の人がレビューすることになっている。そのPull Requestでもらうコメントを見ると「気が狂った設計」「クソコード」「有り得ない」と"記録に残るパワハラ。余裕を作り出さない限り改善しないだろうな。
hdampty7
自分も言われるのは嫌だけどクソすぎると全部レビューするくらいなら最初から自分で書き直した方が良いと思うことが多々ある。実際には丁寧にやさしく申し訳なさそうに修正してもらうように提案してるけど。
redmagic1417
PRだろうがなんだろうが、社内における対人コミュニケーションなわけで、それを文化として許容するなんて相当ひどい職場だと思うが。実際コードに問題があった場合ですら、こんな職場はひどい。学生サークルかよ。
koheisg
増田主は罵り言葉が使われなければ、問題ないのかな？うちのPJはmust imo nitsの３段階で対応優先度を表してるけど、MUSTが多く並ぶとどんなに柔らかい言葉でもそれなりに辛いし、レビュワーな時はMUSTがある程度並ぶと話に
vanbraam
b:id:entry:258250548より;まず理由を示していない段階で"気が狂った設計","クソコード","有り得ない"はクソコメント;Linusの例は「レビュー依頼が多すぎる」背景があるので,有象無象のレビューを拒否したい時以外は意味がない
oakbow
コード扱き下ろすのが開発者の文化と思っている人がなぜか結構いるようだけど、そんなことないよ。そのくらいの方が却って以心伝心な環境もあるとは思うけど、増田はそうなってないわけだし言うべきだと思う。
june29
レビューの目的が「コードがベターな状態に向かうように促すこと」だとしたら、この人は「よーし、やるぞ〜」と思えていないわけだし、目的未達成で、レビューする人の行動を変えることを考えた方がよいと思う。
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
いまの話題をアプリでチェック！
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
関連記事
Pull Requestのレビューが辛くて会社をやめたい
私はプログラマに向いていないのかもしれない。 うちのチームではコミットをmasterブランチへマージする... 私はプログラマに向いていないのかもしれない。 うちのチームではコミットをmasterブランチへマージする前に、Pull Requestを出してそれをリーダーや他の人がレビューすることになっている。（たぶんよくあるフロー？） そのPull Requestでもらうコメントを見ると死にたくなってくる。特に辛い気持ちになるのが、「気が狂った設計」「クソコード」「（こんな実装は）有り得ない」といった言葉だ。 レビューにおいてそういった強い言葉がときとして必要なことは理解している（そういえばこなものもあったなと思い出した http://cpplover.blogspot.jp/2013/07/linuxml.html ）。またそういったコメントを残す相手との仲が険悪なわけでもない。またよいと思ってくれたらしいところは褒めてくれたりLGTMしてくれたりもする。 だけどそれでも辛い。きつい言葉を向けられる
ブックマークしたユーザー
すべてのユーザーの
詳細を表示します
詳細を表示します
2015/07/03 リンク