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