プロジェクト

全般

プロフィール

Pull Request Policy (ja) » 履歴 » バージョン 1

kaoru n, 2014-10-21 13:19

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
h3. プルリクエスト方法
24
25
https://github.com/openpne/OpenPNE3/pulls
26
上記URLより「New pull request」ボタンを押下して、プルリクエストを行ってください。
27
タイトルは下記の例のように、Redmine 側のチケットのタイトルと合わせ、チケットの番号を含めてください。
28
例)Bugチケットの場合: 
29
<pre>
30
(refs #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある
31
</pre>
32
例)Backportチケットの場合: 
33
<pre>
34
(refs #3562 BP from #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある
35
</pre>
36
もし、Redmineにまだチケットが作成されていない場合は、新規に作成してください。
37
チケットを発行する場合は、 "How To Report Issue (ja)":https://redmine.openpne.jp/projects/op3/wiki/How_To_Report_Issue_%28ja%29 を参照してください。
38
githubでのプルリクエスト方法については、"こちら":https://help.github.com/articles/creating-a-pull-request を参照してください。
39
40
h3. チケットでの報告
41
42
プルリクエスト後、チケットのコメントにプルリクエストのURLを記入し投稿してください。
43
44
h3. コードレビュー・テストでのフィードバックへの対応
45
46
コードレビューまたはテストでのフィードバックは、チケットを通して行われます。
47
※プルリクエストのソースに直接コメントが有る場合は、チケットにもその旨が記入されます。
48
開発者はフィードバックに対応後、チケットのステータスを「Pending Review(レビュー待ち)」に変更してください。