Pull Request Policy (ja) » 履歴 » バージョン 1
kaoru n, 2014-10-21 13:19
1 | 1 | kaoru n | h1. プルリクエストにおけるポリシー |
---|---|---|---|
2 | 1 | kaoru n | |
3 | 1 | kaoru n | h3. プルリクエストに含まれるソースコードについて |
4 | 1 | kaoru n | |
5 | 1 | kaoru n | 対応チケットへの変更分のみを含んでください。 |
6 | 1 | kaoru n | 以下のコミットメッセージのルール、OpenPNE3 PHP 標準コーディング規約、OpenPNE3 セキュアコーディングガイドラインにも準拠してください。 |
7 | 1 | kaoru n | * "コミットメッセージのルール":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 | 1 | kaoru n | * "OpenPNE3 PHP 標準コーディング規約":http://www.openpne.jp/coding-standards-ja/ |
9 | 1 | kaoru n | * "OpenPNE3 セキュアコーディングガイドライン":http://www.openpne.jp/developer/secure-coding-guideline/ |
10 | 1 | kaoru n | |
11 | 1 | kaoru n | また、必須ではありませんが、テストコードが含まれていて実際に動作しテストが成功するコードであることを目指してください。 |
12 | 1 | kaoru n | |
13 | 1 | kaoru n | h3. プルリクエストのbaseとして指定するブランチ |
14 | 1 | kaoru n | |
15 | 1 | kaoru n | <pre> |
16 | 1 | kaoru n | https://github.com/openpne/OpenPNE3/tree/master |
17 | 1 | kaoru n | </pre> |
18 | 1 | kaoru n | |
19 | 1 | kaoru n | h3. プルリクエストのcompareとして指定するブランチ |
20 | 1 | kaoru n | |
21 | 1 | kaoru n | 開発者のリポジトリの作業用のブランチを指定してください。 |
22 | 1 | kaoru n | |
23 | 1 | kaoru n | h3. プルリクエスト方法 |
24 | 1 | kaoru n | |
25 | 1 | kaoru n | https://github.com/openpne/OpenPNE3/pulls |
26 | 1 | kaoru n | 上記URLより「New pull request」ボタンを押下して、プルリクエストを行ってください。 |
27 | 1 | kaoru n | タイトルは下記の例のように、Redmine 側のチケットのタイトルと合わせ、チケットの番号を含めてください。 |
28 | 1 | kaoru n | 例)Bugチケットの場合: |
29 | 1 | kaoru n | <pre> |
30 | 1 | kaoru n | (refs #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある |
31 | 1 | kaoru n | </pre> |
32 | 1 | kaoru n | 例)Backportチケットの場合: |
33 | 1 | kaoru n | <pre> |
34 | 1 | kaoru n | (refs #3562 BP from #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある |
35 | 1 | kaoru n | </pre> |
36 | 1 | kaoru n | もし、Redmineにまだチケットが作成されていない場合は、新規に作成してください。 |
37 | 1 | kaoru n | チケットを発行する場合は、 "How To Report Issue (ja)":https://redmine.openpne.jp/projects/op3/wiki/How_To_Report_Issue_%28ja%29 を参照してください。 |
38 | 1 | kaoru n | githubでのプルリクエスト方法については、"こちら":https://help.github.com/articles/creating-a-pull-request を参照してください。 |
39 | 1 | kaoru n | |
40 | 1 | kaoru n | h3. チケットでの報告 |
41 | 1 | kaoru n | |
42 | 1 | kaoru n | プルリクエスト後、チケットのコメントにプルリクエストのURLを記入し投稿してください。 |
43 | 1 | kaoru n | |
44 | 1 | kaoru n | h3. コードレビュー・テストでのフィードバックへの対応 |
45 | 1 | kaoru n | |
46 | 1 | kaoru n | コードレビューまたはテストでのフィードバックは、チケットを通して行われます。 |
47 | 1 | kaoru n | ※プルリクエストのソースに直接コメントが有る場合は、チケットにもその旨が記入されます。 |
48 | 1 | kaoru n | 開発者はフィードバックに対応後、チケットのステータスを「Pending Review(レビュー待ち)」に変更してください。 |