Pull Requestのレビューが辛くて会社をやめたい

私はプログラマに向いていないのかもしれない。 うちのチームではコミットをmasterブランチへマージする前に、Pull Requestを出してそれをリーダーや他の人がレビューすることになっている。（たぶんよくあるフロー？） そのPull Requestでもらうコメントを見ると死にたくなってくる。特に辛い気持ちになるのが、「気が狂った設計」「クソコード」「（こんな実装は）有り得ない」といった言葉だ。 レビューにおいてそういった強い言葉がときとして必要なことは理解している（そういえばこなものもあったなと思い出した http://cpplover.blogspot.jp/2013/07/linuxml.html ）。またそういったコメントを残す相手との仲が険悪なわけでもない。またよいと思ってくれたらしいところは褒めてくれたりLGTMしてくれたりもする。 だけどそれでも辛い。きつい言葉を向けられる

[windish]

チーム開発でチームの雰囲気を悪くする人はどんなにコードが書けても一緒にお仕事したくないです。/ 辛いのは周りに伝えよう。マジで伝わってない時があるので！

[t-wada]

これはレビュアーが未熟すぎるのだと思う

[nekoprotocol]

言おう！

[sho]

チームで「Team Geek」を読むことをおすすめする http://www.amazon.co.jp/dp/4873116309/?tag=cshs-22

[kuenishi]

Githubがどうこう以前の人間プロトコルの問題

[nullpobug]

なぜダメなのか、どうすべきなのか、が書かれてない否定のレビューコメントは、クソ。上司に言ったほうがよい。

[girled]

エンジニアはコミュ症でもしょうがないっていう空気がありがちだけどでも「本当に出来る人はコミュ充」という現実も事実。ってみんなが認識できると幸せになるかもだよなあ。

[Kwappa]

【宣伝】これの第7章をおすすめします！ http://www.amazon.co.jp/dp/B00QPSXY1I/ref=cm_sw_r_tw_awdl_MwvLvb0JMTVJ4

[koyancya]

急に泣き出してみるのはどうか

[ch1248]

ネコプさんも言ってるように、「辛い」って気持ちを伝えたり、相談した方が良いと思う。

[vamview]

“LGTM”

[nabetama]

上司に即言った方がいい

[rlho]

めちゃくちゃわかる “よいプロダクトを作る上で強い言葉を交えた議論が必要不可欠ならば、それに強いストレスを感じてしまう私はプログラマとして適正がないのでは？”

[hiroomi]

"リーダーや他の人がレビューすることになっている。そのPull Requestでもらうコメントを見ると「気が狂った設計」「クソコード」「有り得ない」と"記録に残るパワハラ。余裕を作り出さない限り改善しないだろうな。

[morygonzalez]

レビューは事実だけを指摘するべきだし「気が狂ってる」とかは仮に狂ったコードだったとしても書いてはいけないと思う。「あり得ない」とか「意味がわからない」とかも NG

[takeda25]

ぼくでもこんなことは書かないなぁ

[hdampty7]

自分も言われるのは嫌だけどクソすぎると全部レビューするくらいなら最初から自分で書き直した方が良いと思うことが多々ある。実際には丁寧にやさしく申し訳なさそうに修正してもらうように提案してるけど。

[assaulter]

あまり強い言葉を使うと弱く見えるんだよなぁ

[hyujico]

いや、そんな主観的なコメント必要ないでしょ。必要なのは問題点の指摘と改善案の提案のみ。レビュアーの程度が低すぎる。そういう文化の会社なら転職するのも一つの選択肢だと思うわ。まじで

[Pasta-K]

理由も何もなしに書いてあるのだとしたら、そのレビュアーがただそういうことを言いたいだけなのでは

[megomego]

会社やめたくなる程のレビューだとしたら、社内にレビュー以前の問題があるとしか思えない

[redmagic1417]

PRだろうがなんだろうが、社内における対人コミュニケーションなわけで、それを文化として許容するなんて相当ひどい職場だと思うが。実際コードに問題があった場合ですら、こんな職場はひどい。学生サークルかよ。

[gabari]

悪いものは悪いと言わないとなぁなぁで進むので、難しいところではある。

[ahmok]

コード書く人の世界に罵倒するとことかあるとか初めて知った。仕事変えずに相手変えた方が良いような…

[koheisg]

増田主は罵り言葉が使われなければ、問題ないのかな？うちのPJはmust imo nitsの３段階で対応優先度を表してるけど、MUSTが多く並ぶとどんなに柔らかい言葉でもそれなりに辛いし、レビュワーな時はMUSTがある程度並ぶと話に

[dev0000_1]

レビューがただのDisになっているというか、ラップだとMCバトルが始まるな。

[vanbraam]

b:id:entry:258250548より;まず理由を示していない段階で"気が狂った設計","クソコード","有り得ない"はクソコメント;Linusの例は「レビュー依頼が多すぎる」背景があるので,有象無象のレビューを拒否したい時以外は意味がない

[amnesia4646]

こんなレビューもらったらレスの文末に全部 :hocho: って入れてしまいそう…

[rainbow_doll]

その工数と指示じゃクソコード量産になりますけどいいですか？って事前にPMに聞いてやってる。殆どの場合、いいよー動けばで済むから楽。保守する人はごめんなさい。

[shuzo_kino]

血に餓えたWEB荒野案件。ほかの人も言ってるけどTeam Geekを処方したいレベル

[takun71]

まあ文章に起こすときつくなるってことあるよね。論争になりそうだったらレビュワーと直接話した方がいいと思う

[oakbow]

コード扱き下ろすのが開発者の文化と思っている人がなぜか結構いるようだけど、そんなことないよ。そのくらいの方が却って以心伝心な環境もあるとは思うけど、増田はそうなってないわけだし言うべきだと思う。

[paulownia]

「気が狂った設計」「クソコード」「（こんな実装は）有り得ない」これは感想であってレビューじゃないです。だめ。コードの問題点を説明するのがレビュアーの仕事です

[fjwr38]

クソ設計 / 実装だったとしても、管理とかチームマネジメントの問題だと思うんだよなぁ。設計とか実装になる前に何かしらフィードバックとかしたほうが建設的じゃんねえ。

[june29]

レビューの目的が「コードがベターな状態に向かうように促すこと」だとしたら、この人は「よーし、やるぞ〜」と思えていないわけだし、目的未達成で、レビューする人の行動を変えることを考えた方がよいと思う。

[aliliput]

これはどこの業界でもそう。如何なる場合でもひどい言葉を使う必要はない。悪しき慣習だと思う

[todesking]

ワハハ(´･_･`)

[ngmy]

？？？「…あまり強い言葉を遣うなよ」

[akitsukada]

レビュアーのレベルが低いだけでは