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