「[[.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

トップ   編集 差分 バックアップ 添付 複製 名前変更 リロード   新規 一覧 単語検索 最終更新   ヘルプ   最終更新のRSS