アラフォーからのプログラミングとデザイン

大人から始めたプログラミングとデザインについてのあれこれ

開発日記vol/10:プルリクエストは細かく出す

f:id:simpledancer:20180228113608j:plain

プルリクエストとは

プルリクエストとは簡単に言うと、開発者のローカルリポジトリでの変更を他の開発者に通知する機能です。プルリクエストは次のような機能を提供します。

  • 機能追加や改修など、作業内容をレビュー・マージ担当者やその他関係者に通知します。
  • ソースコードの変更箇所をわかりやすく表示します。
  • ソースコードに関するコミュニケーションの場を提供します。

backlog.com

私は今開発を個人でもやっていて、メンターにレビューをしてもらってるんですが

ここ最近

できれば細かくプルリクエスト出してください。。。!と言われてしまいまして

個人的には、まとめて出した方が手間がかからないんじゃないか?と思ってたんですけどどうやら違うようです。

simpledancer.hatenablog.com

レビュアーにとっては結構めんどい(多分)

私みたいなペーペーだと大したコードなんて書けないけど

変更が大量になったり一個のPRにたくさん会話がつくと読みにくくなるらしい

そもそも一気に変更したのをPR出してしまうとレビュー自体がめちゃ大変なんだそうです。

techracho.bpsinc.jp

そんなわけで気をつけながら早めにPRを出そうとしたら今度はプルリクエスト出せない問題が出てきました。

baseになるものとcompareするものをいつも通りに当てはめたら

(base→master compare→他ブランチ)

There is anything compareとなってしまったのですね。色々試行錯誤してローカルで書いたブランチをpushしているにも関わらず!なぜだろう。。

解決したらまた書きます。