Skip to content

Review и merge

После подготовки sync/* ветки обновление шаблона должно попасть в master через PR/MR.

Создать PR/MR

Параметры:

text
source: sync/update-template-vX
target: master

Цель review — увидеть:

  • какие изменения пришли из шаблона;
  • какие конфликтные решения были сделаны в sync/*;
  • не попали ли в обновление лишние изменения приложения;
  • проходят ли проверки проекта.

Настройки merge

Для sync-PR/MR важно:

text
squash = off
fast-forward merge = хорошо
merge commit       = допустимо
squash merge       = нельзя

Squash нельзя использовать, потому что он может уничтожить связь истории master с историей template. Особенно это критично после миграционного sync/bootstrap-template.

Проверки перед merge

Проверьте граф истории:

bash
git --no-pager log --oneline --graph --decorate --all --max-count=50

Проверьте итоговый diff:

bash
git --no-pager diff origin/master...sync/update-template-vX

Проверьте проект обычными командами конкретного приложения, например:

bash
npm run lint
npm run build

После merge

После успешного merge в master можно удалить временную ветку:

bash
git branch -d sync/update-template-vX
git push origin --delete sync/update-template-vX

Проверьте, что master теперь содержит обновление шаблона:

bash
git fetch origin
git --no-pager log origin/template..origin/master --oneline
git --no-pager diff origin/template...origin/master