Yii2 framework backup
You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
 
 
 
 
 

757 B

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

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

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