開発速度を上げるための Pull-Request のつくり方 - クックパッド開発者ブログ

こんにちは、投稿開発部の森川 (@morishin127) です。クックパッド、お料理アルバム、みんなのお弁当の iOS アプリの開発等に携わっています。 クックパッドでの開発は GitHub Enterprise 上で行われており、書いたコードをプロダクトに取り込む前には基本的に第三者のコードレビューが必須です。コードレビューはプロダクトの品質向上に貢献していますが、往々にして結構な時間と労力がかかるものです。Pull-Request を出してレビューをしてもらい指摘の修正を繰り返していると、場合によってはマージに数日〜1週間ほどかかってしまうこともあります。自分の開発速度を速めるため、また周りのエンジニアの開発速度を下げないためにレビューしやすい Pull-Request を出すことは重要です。この記事ではレビューしやすい Pull-Request のために心がけていることを紹介したい

[june29]

ぼくがレビューを依頼されるときはだいたいここに書いてあるようなことが守られていて、いつもレビューしやすくて助かります。依頼する側が工夫する方がトータルでコストが安く済むと思っている派です。

[tchssk]

レビュアーの負荷は大きくなりがちなので、レビュイーが気をつかう文化ができてるのが素晴らしい。

[efcl]

PRのスクショTips よくmonosnapで撮って、gifbreweryでGIF化してる

[sue445]

修正前後のスクショを横に並べるのむっちゃよい

[koogawa]

ためになる / diff 多すぎ問題はホント気を付けたい🙇🏻

[kuborn]

やるかやらないかは別としてもっといろんなところの運用方法が知りたいので、こういうのすごく嬉しい。自分も発信しなきゃな。

[yokoyantech]

よいプルリクとはなんなのか参考になります。UIを並べるのいいですね。

[mosco-mule]

開発速度を上げるための Pull-Request のつくり方 こんにちは、投稿開発部の森川 (@morishin127) です。クックパッド、お料理アルバム、みんなのお弁当の iOS アプリの開発等に携わっています。 Tags: 2 minutes from Pocket August 18, 2016 at

[minifigures]

ぼくがレビューを依頼されるときはだいたいここに書いてあるようなことが守られていて、いつもレビューしやすくて助かります。

[sekitaka_1214]

あとでよむ

[peroon]

“シュッとした感じ” 良い

[Haaaa_N]

モバイルアプリか,ならスクリーンショットは少し大変ですね

[NineGates]

開発速度を上げるための Pull-Request のつくり方 こんにちは、投稿開発部の森川 (@morishin127) です。クックパッド、お料理アルバム、みんなのお弁当の iOS アプリの開発等に携わっています。 Tags: from Pocket August 19, 2016 at 09:09AM vi

[ofsilvers]

よい

[slippyvalley]

わかりやすい

[koheisg]

pull requestを上手くやる工夫大事

[k1take]

「アニメーション GIF で動作確認」動画って有用だよね。プレゼンとかでも。

[inazakira]

500行

[kazuph1986]

gifはLICEcapを社内では使ってた。

[braitom]

iOSアプリ開発におけるPRの方法。UIの変更前後のスクリーンショット貼ったり、GIF使って動作が分かるようにしたり。

[chanohajp]

今度使おう

[sitk]

メモ。

[tbpg]

画像やアニGIFよい

[s12bt]

アニメーションGIFという手があるか

[kakku22]

PR 分割，UI 変更時の事前事後キャプチャなど，自分だともう癖になってしまってるけど，チーム全体で徹底できるようになるとプロセス効率が上がるね

[aruiha_shikashi]

"数百行以内に収まるように分割" 分割の仕方が知りたい。その変更だけを入れてリリースしても問題ないような部分へ分割できれば良いだろうけど、分割したらtestどころかcompileも通らない、ってことありません？

[jhuyiyfd]

やったこととかを分けて書いてみては！

[andoshin11]

バズしん案件だ

[koheyhey18]

:morishin:

[takatama]

PRコメントは目的とやったことを分けて書く

[innx_hidenori]

カルテットでも自然とこの記事とほぼ同じになってる

[norip44]

キャプチャ撮るのは真似します。