プロジェクト

全般

プロフィール

編集 履歴

Wiki »

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

Youichi Kimura, 2016-08-09 15:01

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
    3
    Youichi Kimura
    
@openpne/OpenPNE3@ の Milestone 一覧: https://github.com/openpne/OpenPNE3/milestones





    32
    
    
    





    33
    1
    kaoru n
    
h3. プルリクエスト方法





    34
    
    
    





    35
    
    
    
https://github.com/openpne/OpenPNE3/pulls





    36
    
    
    
上記URLより「New pull request」ボタンを押下して、プルリクエストを行ってください。





    37
    
    
    
タイトルは下記の例のように、Redmine 側のチケットのタイトルと合わせ、チケットの番号を含めてください。





    38
    
    
    
例）Bugチケットの場合: 





    39
    
    
    
<pre>





    40
    
    
    
(refs #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある





    41
    
    
    
</pre>





    42
    
    
    
例）Backportチケットの場合: 





    43
    
    
    
<pre>





    44
    
    
    
(refs #3562 BP from #3560) スマートフォンで画像をアップロードすると画像が横向きになってしまう場合がある





    45
    
    
    
</pre>





    46
    
    
    
もし、Redmineにまだチケットが作成されていない場合は、新規に作成してください。





    47
    
    
    
チケットを発行する場合は、 "How To Report Issue (ja)":https://redmine.openpne.jp/projects/op3/wiki/How_To_Report_Issue_%28ja%29 を参照してください。





    48
    
    
    
githubでのプルリクエスト方法については、"こちら":https://help.github.com/articles/creating-a-pull-request を参照してください。





    49
    
    
    





    50
    
    
    
h3. チケットでの報告





    51
    
    
    





    52
    
    
    
プルリクエスト後、チケットのコメントにプルリクエストのURLを記入し投稿してください。





    53
    
    
    





    54
    
    
    
h3. コードレビュー・テストでのフィードバックへの対応





    55
    
    
    





    56
    
    
    
コードレビューまたはテストでのフィードバックは、チケットを通して行われます。





    57
    
    
    
※プルリクエストのソースに直接コメントが有る場合は、チケットにもその旨が記入されます。





    58
    
    
    
開発者はフィードバックに対応後、チケットのステータスを「Pending Review（レビュー待ち）」に変更してください。