Перейти к содержимому

Контроль качества pull request

При проверке PR на возможность слияния нужно учитывать следующие критерии:

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