「[[.NET 開発基盤部会 Wiki>http://dotnetdevelopmentinfrastructure.osscons.jp]]」は、「[[Open棟梁Project>https://github.com/OpenTouryoProject/]]」,「[[OSSコンソーシアム .NET開発基盤部会>https://www.osscons.jp/dotNetDevelopmentInfrastructure/]]」によって運営されています。 -[[戻る>Githubの使い方]] *目次 [#l56ca360] #contents *概要 [#re6e444a] *Pull Request一覧 [#v0388a95] **Filters [#kcd588c5] [[Issue>Github - Issue、Label、Milestone関連]]と同様にFiltersを実行できる。 既定で以下が適用されている。 is:pr is:open -ソート~ ソートを選択してソートできる。 --Newest --Oldest --Most commented --Least commented --Recently updated --Least recently updated 多分、Filters で組み合わせできると思う。 **Label [#k7ba1f15] [[Issue>Github - Issue、Label、Milestone関連]]と同様にLabelを設定できるが、~ Pull requestはIssueに対応付けるので、~ Pull request側にLabelを張らない事が多い。 **Milestone [#pe2d671e] [[Issue>Github - Issue、Label、Milestone関連]]と同様にと同様にMilestoneを設定できる。 *Pull Requestの送信方法 [#ka3b4a35] **Repository間 [#l9adba67] -Fork元のRepositoryに対してPull Requestを送信することができる。 -Public RepositoryとPrivate Repository間ではPull Requestを送信できない([[裏技>#s157068b]])。 **Branch間 [#a3d11fc4] -Branch元のBranchに対してPull Requestを送信することができる。 -Branchする時は、Branch元のブランチを選択・表示させた状態で、~ BranchのDropDownのFind or create a branchに存在しないBranchを入力する。 *Pull Requestの作法 [#rb356a29] **[[CONTRIBUTING.md>#w5998e01]] [#c375c9fc] CONTRIBUTING.mdに記載されている貢献(コントリビュート)の作法に従う。 **featureブランチ [#i3d4dfe9] -Pull Requestを作成にはfeatureブランチを作成する必要があります。 --featureブランチの修正後、都度Pull Requestを送ってレビューを依頼します。 --featureブランチに異なるIssue(問題やエンハンス)のタスクの修正を混ぜてはいけません。 -複数のfeatureブランチ --Pull Requestのサイズが大きくなりレビューが困難になる場合は、 ---複数のfeatureブランチを作成し、複数のブランチ間のPull Requestを送信します。 ---複数のブランチ間のPull Requestに対して、レビューを行います。 --そのブランチで、複数のPull Requestをマージして束ね、 ---Fork元のブランチに集約されたPull Requestを送付します。 ---集約されたPull Requestには、レビューの為の個々のPull Requestのリンクを張ります。 **Pull Requestのサイズ [#efea8628] -レビューし易いサイズに分割~ Pull RequestのレビューのためにPull Requestのサイズを小さくします。 -1 Issue 1~n Pull Request~ 1つのPull Requestの中に複数のIssue(問題やエンハンス)のタスクが存在する場合、~ レビューがボトルネックになるので、問題やエンハンスのタスク毎にPull Requestを作成します。 -不要な修正もコミットしないで下さい。 --IDEやEditorによりインデントが変更されるような修正。 --誤った修正を再度修正するような修正(スラッシングを避ける)。 **Pull Requestの送信前 [#g731fb40] Pull Requestの送信前に(若しくはPushする前に)、~ Pull Requestを送る先のブランチをPullする。 -より良いプルリクエストのための10のヒント | Yakst~ https://yakst.com/ja/posts/1625 --Pull Requestが古くなってきた時にスラッシングが起きる(長い議論のためであることが多い)。 **Pull RequestでIssueをClose [#jcc81140] -Closing issues via commit messages - GitHub Help~ https://help.github.com/articles/closing-issues-via-commit-messages/ -Output_Log: GitHubでコミットとissue(課題)をリンクさせる方法~ http://outputlog.blogspot.jp/2011/04/githubissue.html -Git のコミットメッセージから GitHub 上の Issue を閉じる | 雪猫 ...~ http://blog.snowcait.info/2014/09/22/closing-issues-via-commit-messages-on-github/ *Pull Requestの送信後 [#bf0c53a4] **Pull Request送信後の送信元リポジトリの更新 [#af234dc6] Pull Request送信後に送信元リポジトリを更新した場合の動作は、 -Pull Requestに更新が反映される。 -Files changedに結果が反映されるには時間差がある。 となります。 従って、 -(Files changedに反映されていないだけなので、)そのまま、mergeしてもよし、 -レビュー等を急いでいるときはCloseして、再度、Pull Requestを投げてもよし。 です。 **[[Pull Requestのコンフリクトの解消]] [#l116732e] *その他 [#eff8a1bd] **[[PublicとPrivate Repository間でPull Requestを送信する。]] [#s157068b] **[["Bump handlebars from X to Y in Z" と言う PR]] [#r1a6e7da] **[['Bump handlebars from X to Y in Z' と言う PR]] [#r1a6e7da] *参考 [#r34cc798] -より良いプルリクエストのための10のヒント | Yakst~ https://yakst.com/ja/posts/1625 -[初心者向け]こんなときどうする⁉︎ GitのTips25選! - Sweet Escape~ http://keisuke69.hatenablog.jp/entry/2013/12/16/130110 -Gitでやらかした時に使える19個の奥義 - Qiita~ http://qiita.com/muran001/items/dea2bbbaea1260098051