Контроль качества pull request
При проверке PR на возможность слияния нужно учитывать следующие критерии:
- К PR должна быть привязана issue, либо в PR должно быть понятное описание того, что исправляется или добавляется.
- Модульные тесты. Не обязательно, но приветствуется. Тесты должны падать без кода, который PR исправляет.
- Запись в CHANGELOG. Добавляется в секцию следующего релиза, сортируется по типу issue и номеру. Никнеймы ответственных должны быть указаны.
- Стиль кода и стиль кода представлений должны соблюдаться. Их можно поправить при слиянии, если тот, кто сливает, предпочитает так делать.