初めに
こんにちは！taisei_codeです。
GitHubを使用したチーム開発で私が初めてプルリクエストを提出して、レビュワーからFB（フィードバック）をいただいてから意識していることを書いていきます。
FBと意識すること
作業単位でコミットする
1つのissueを対応できたら、コミットしていたので、issueを細分化して小さな作業単位でコミットするように改善しました。具体的には、issueのゴールが「RSpecを導入してModelスペックがバリデーションを通ることを確認する」であれば、以下のように作業を小さくしてコミットします。
① インストールするGemを追加
② 自動生成されるファイルを編集
③ テストコードを書く
PRのタイトルを作業内容に変更する
変更前は、技術的な問題で解決したいことをPRのタイトルにしていました。作業内容というのは、関連するissueで解決したい目的のこと。
詳細項目はレビュワーへの共有事項をメインに使う
私は詳細項目に質問フォーマットを共有していてご指摘を受けました。詳細項目には、プルリクエストの作成者が変更の内容を詳細に共有することに使われるので、技術的に質問があるときはSlackでやり取りするようにします。
終わりに
レビュワーからいただいたFBは直ぐに改善できる内容です。最初はプルリクエストに慣れませんが、1つずつ整理しながら作業していくことが大切だと感じました。