ログイン - はてな

パスワードを忘れた方はパスワードの再設定を行ってください。 初めての方ははてなID登録 (無料) してください。 うまくログインできない方はお問い合わせをご覧いただき、Cookieの設定をご確認ください。

[teckl]

「レビューしにくいPRは…最悪の場合はノールックLGTMされてバグの温床になる」ｗｗ

[hush_puppy]

他人のコードのレビューはかなり上位の優先事項にしてる。すぐに開始したことを伝える。すぐにできないときでも数分以内に伝える。レビューに限らず、他人と仕事をするときは、レイテンシを重視してる。

[cardboarder]

いい話なのでPULL_REQUEST_TEMPLATE.mdでテンプレ化すればよさそう。

[rjge]

よいお話。変更内容からじゃ背景や目的は推測しかできないしそれをいちいち確認するのも面倒だからせめてwhyは書いてほしい。

[uskey]

“基本的にレビューとマージのサイクルを早くすることで、PR間の依存が深くならないようにしましょう。”最後はぜったいここに落ち着くからううむという感がある

[PEEE]

後で背景を追いやすいのでチームみんなこういう風に考えていればマジで楽になる予感。でも真面目にやる人少ないんだよな…

[mosco-mule]

レビューしてもらいやすいPRの書き方 こんにちは。Androidエンジニアの黒川（@hydrakecat）です。 みなさん、Pull Request（PR）は書いているでしょうか？ PRを出したあとの1つの悩みに、なかなかレビューしてもらえないというも

[iga_k]

良いまとめ

[vanbraam]

"差分だけでなくコミット単位でレビューする人"<自分もそう;"コードを読んだだけでは理解できない/しにくい事"については目的を明記して(≒whyを書いて)ほしい;テストがない場合は確認手順必須だがまずテストを書くべき

[efcl]

レビューしやすいPRについて

[kosukem]

コードレビュー

[braitom]

レビュワーにやさしいPRの書き方が分かりやすくまとまっている。

[shunt_i]

プルリク

[murakaming]

自分でPRの差分にコメント書くの凄い良いけど、言い訳したいだけにならないよう気をつける必要がある

[Nyoho]

pull request の how-to. 確認手順とかきちっと書くと受ける方はわかりやすくていいね。

[h5y1m141]

自分がこれまでお手伝いしてきた所はたいてい、過去の経緯わからないのが多く、反面教師として将来誰かが自分のPR見た時にこまらないように細かく説明書いてる

[restartr]

良い提案。関連PRとレビュー期限も明記されているとなお👍かと。

[suginoy]

“ぜひ、自分のかわいい成果を売り込むことを考えて、レビューしやすいPRを書いてください。”

[wkubota]

このへんの書き方にそって楽して出したいんだよね。プルリクって。

[wyukawa]

確認手順は書いたこと無かった。

[nn0]

ちゃんとPRの内容書かない人って、面倒くさがってて品質も悪い気がする。同じ指摘何度も受けたり。

[to4iki]

思いやりだ

[trollvinter]

これは他のことにも応用できそうな話ですね

[windish]

確認手順を書くのはよいな。今度やってみよう

[ujihisa]

pullreqはサイズの二乗に比例して難易度があがる

[reitanigawa]

わかる話なのだが、肥大化するｗ

[call_me_nots]

“レビュワーからすると、一般に差分が小さいほどレビューは容易になります。 経験的には、差分が大きくなると、二次関数的にレビューの困難さが増す気がします”

[himanoa]

心がけていきたい

[mizoguche]

いい話

[koogawa]

尊い

[tomoima525]

良記事！テンプレ作るのも良い手段ですよね