なぜあなたのPull Requestは読まれないのか - Qiita

Pull Requestを出してレビューしてもらってから反映。 どこにでもあるありふれた開発フローに付きまとう、どこにでもあるありふれた問題。 「Pull Requestがレビューされない」 もちろん開発フローにレビューが含まれている以上、レビューをしないメンバーにも非がないとは言えませんが、多くの場合はレビューされないPRには問題があるものです。 デカい 兎にも角にもデカいPRは読むのがつらいです。 もちろん要件が明記されていないなど、他にもPRが読みにくくなる原因はたくさんありますが、一番はこれです。 極端な話、1行変更のPRは他に何も書かれていなくても実装内容を察することができますが、10ファイル100行の差分と箇条書き20点の要件が書かれたPRは内容を把握するだけで一苦労です。 しかし、このこと自体は数カ月でもコードを書いていれば自然と勘づくもの。 問題はなぜPRが大きくなってしま

[luccafort]

“Issue = PRではないため”非常によくわかるんだけどぼくはこの問題はIssueがデカすぎるんだという判断で逆にIssueをより細かくわけて対応してるけどな。

[diveintounlimit]

正しいけど、ウチの場合はこれ理想論だなぁ。メンバーに機能単位でと言いつつ「ぼくのかんがえた」機能単位でPR切る人いるけど、ちゃんと分けるとPR数2倍くらいに増えそうだし、変な手間が増えてたぶん回らない（2階へ

[hyujico]

一人プロジェクトで、誰にもレビューされないのでPRすることがなくて泣いた。チケットの単位を細かくして、1チケット、1ブランチ、1コミットでプッシュしてます。一人プロジェクトなので誰も見ることないけど

[hush_puppy]

コードレビューはどんなものでも、すぐに返事して、すぐに読むようにしてるな。出す方は、他人の速度には期待せず、小さいのをどんどん出してる。レビューは軽い気持ちで。官僚的にならないように注意してる。

[masaru_b_cl]

“Issueの単位とPRの単位は違う”

[hatsu_mi]

ボーイスカウトするなって話なんだろうか

[nilab]

なぜあなたのPull Requestは読まれないのか - Qiita

[kmizushima]

IssueとPRの関係についてだけは同意できない。というより、IssueとPRが一対一対応するPRが結構あると思う。その証拠に、GitHubではFix #xx と書いたPRがマージされると該当Issueがcloseされるという機能があるわけで。

[joker1007]

いい事を書いている。

[yamadar]

まさにこれで悩んでたので有り難い記事

[braitom]

Pull Requestの粒度の話。Issue、Commitｔの関係性を意識、branch名をしっかりつける、要件をきちんと書くなどを行ってPRのサイズを小さく保つ。

[kikiki-kiki]

大きな要望は要望をまとめたマイルストーンかissueを作って機能を切り分けた小さきissue紐付けて作成して機能別のissueに対して1PR。マージされたら機能別のissueをclose派。PRに関する議論は該当するissueで確認できるから。

[kenichi_odo]

レビューする側としてはPR小さい方がいいだろうけどそうするとブランチだらけになっちゃうのかなっ笑

[akatakun]

メンバー少ないから許して((((；ﾟДﾟ))))ｶﾞｸｶﾞｸﾌﾞﾙﾌﾞﾙ

[xlc]

趣味で仕事されても困る。

[nlogn]

“同じタイミングでリリースしたい機能群については、それを乗せるためのbaseとなるようなPRを作り” そのPRは機能分割されていないように思えるので何が何だか。

[K-Ono]

なんでPull「引っ張る」なんだろーなー（そこからかおまえ)

[ttop]

分からんでもないが、なかなか難しい

[ymm1x]

PR がマージされたら同時に対応する issue がクローズされるようにしているので自分も issue のサイズの方を調節する派

[bb_river]

誰かもイシューから始めろと言ってたからな

[zumi99]

Pull Request

[morototo]

成る程。PRが1ヶ月放置された理由がわかった気がした

[koogawa]

思いやり大事

[yug1224]

デカい

[igrep]

つらい

[mooonymann]

“同じタイミングでリリースしたい機能群については、それを乗せるためのbaseとなるようなPRを作り、そこから各機能を実装するためのbranchを切っています。”

[hoisjp]

タイトルが良い

[wyukawa]

OSSプロダクトでPRが放置される話かと思ったら違った。

[peketamin]

うおおあ、なるほどです…issueの時点でprの粒度を粗方イメージおけばいいのかな…

[fujii_yuji]

エンジニアじゃないしコード読めないけど、納得。一緒に働くエンジニアにはこういう風にやってくれるようにお願いしよう。

[kamipo]

(ﾟＡﾟ;)ｺﾞｸﾘ