Pull Request Policy (ja) » 履歴 » バージョン 2
Youichi Kimura, 2016-08-09 15:00
Milestoneについての説明を追加
| 1 | 1 | kaoru n | h1. プルリクエストにおけるポリシー |
|---|---|---|---|
| 2 | |||
| 3 | h3. プルリクエストに含まれるソースコードについて |
||
| 4 | |||
| 5 | 対応チケットへの変更分のみを含んでください。 |
||
| 6 | 以下のコミットメッセージのルール、OpenPNE3 PHP 標準コーディング規約、OpenPNE3 セキュアコーディングガイドラインにも準拠してください。 |
||
| 7 | * "コミットメッセージのルール":https://redmine.openpne.jp/projects/op3/wiki/Rule_of_Shared_Repository_%28ja%29#%E3%82%B3%E3%83%9F%E3%83%83%E3%83%88%E3%83%A1%E3%83%83%E3%82%BB%E3%83%BC%E3%82%B8 |
||
| 8 | * "OpenPNE3 PHP 標準コーディング規約":http://www.openpne.jp/coding-standards-ja/ |
||
| 9 | * "OpenPNE3 セキュアコーディングガイドライン":http://www.openpne.jp/developer/secure-coding-guideline/ |
||
| 10 | |||
| 11 | また、必須ではありませんが、テストコードが含まれていて実際に動作しテストが成功するコードであることを目指してください。 |
||
| 12 | |||
| 13 | h3. プルリクエストのbaseとして指定するブランチ |
||
| 14 | |||
| 15 | <pre> |
||
| 16 | https://github.com/openpne/OpenPNE3/tree/master |
||
| 17 | </pre> |
||
| 18 | |||
| 19 | h3. プルリクエストのcompareとして指定するブランチ |
||
| 20 | |||
| 21 | 開発者のリポジトリの作業用のブランチを指定してください。 |
||
| 22 | |||
| 23 | 2 | Youichi Kimura | h3. プルリクエストの Milestone に指定するマイルストーン |
| 24 | |||
| 25 | !{width:354px}pr-set-milestone.png(Pull Request の作成画面で Set milestone のメニューを開いた状態のキャプチャ。master, stable-3.6.x, stable-3.8.x が選択可能な状態になっている)! |
||
| 26 | |||
| 27 | Pull Request の作成時（作成後でも可）に、上図のように Milestone をセットすることができます。 Pull Request のマージ先 (base に指定したブランチ) と同じ名前の Milestone を選択して下さい。 |
||
| 28 | |||
| 29 | Pull Request に Milestone がセットされていない状態でもチケットのワークフロー自体には支障はありませんが、Milestone がセットされていないと Pull Request がそれぞれどのバージョンを対象としたものか確認するためには個々の Pull Request のページを参照しなければならないため、なるべく適切な Milestone がセットされた状態にして下さい。 |
||
| 30 | |||
| 31 | 1 | kaoru n | h3. プルリクエスト方法 |
| 32 | |||
| 33 | https://github.com/openpne/OpenPNE3/pulls |
||
| 34 | 上記URLより「New pull request」ボタンを押下して、プルリクエストを行ってください。 |
||
| 35 | タイトルは下記の例のように、Redmine 側のチケットのタイトルと合わせ、チケットの番号を含めてください。 |
||
| 36 | 例）Bugチケットの場合: |
||
| 37 | <pre> |
||
| 38 | (refs #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある |
||
| 39 | </pre> |
||
| 40 | 例）Backportチケットの場合: |
||
| 41 | <pre> |
||
| 42 | (refs #3562 BP from #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある |
||
| 43 | </pre> |
||
| 44 | もし、Redmineにまだチケットが作成されていない場合は、新規に作成してください。 |
||
| 45 | チケットを発行する場合は、 "How To Report Issue (ja)":https://redmine.openpne.jp/projects/op3/wiki/How_To_Report_Issue_%28ja%29 を参照してください。 |
||
| 46 | githubでのプルリクエスト方法については、"こちら":https://help.github.com/articles/creating-a-pull-request を参照してください。 |
||
| 47 | |||
| 48 | h3. チケットでの報告 |
||
| 49 | |||
| 50 | プルリクエスト後、チケットのコメントにプルリクエストのURLを記入し投稿してください。 |
||
| 51 | |||
| 52 | h3. コードレビュー・テストでのフィードバックへの対応 |
||
| 53 | |||
| 54 | コードレビューまたはテストでのフィードバックは、チケットを通して行われます。 |
||
| 55 | ※プルリクエストのソースに直接コメントが有る場合は、チケットにもその旨が記入されます。 |
||
| 56 | 開発者はフィードバックに対応後、チケットのステータスを「Pending Review（レビュー待ち）」に変更してください。 |