Files
yii2/docs/internals-ja/pull-request-qa.md
2015-10-13 07:55:42 +09:00

757 B

プルリクエストの品質保証

PR をマージするか否かをチェックするときには、特に以下の基準が考慮されるべきです。

  • PR にリンクされている課題(イッシュー)が存在するか、または、PR がどのようなことを修正ないし追加しようとしているのかに関する十分な説明があること。
  • 単体テスト。必須ではありませんが、非常に高く評価されます。PR によって修正されたコードが無ければ失敗する、というテストであること。
  • CHANGELOG のエントリがあること。エントリは次のリリースのセクションに、イッシューのタイプと番号の順に書き入れます。 担当した者のニックネームがあること。
  • コードスタイル および ビューコードスタイル が OK であること。これらは、マージされる際に、マージする者の判断に従って修正される場合があります。