Feature/WMSDK-547 CI/CD#24
Conversation
Vailence
force-pushed
the
feature/WMSDK-547-CI-CD
branch
from
0c53c5d to
4a101a3
Compare
| - name: Merge Pull Request | ||
| run: | | ||
| pr_number=$(gh pr list --base develop --head master --json number --jq '.[0].number') | ||
| gh pr merge $pr_number --merge --auto |
There was a problem hiding this comment.
Он смержится сам? А если упадет?
There was a problem hiding this comment.
Полностью перенесено с iOS-SDK, мы так живем, все нормально)
There was a problem hiding this comment.
Если конфиликтов не будет и / или required checks выполнятся успешно - смерджится. Если что-то пойдет не так из этого, просто не произойдет сам merge и останется висеть открыть PR
| @@ -1,10 +1,10 @@ | |||
| name: "Manual Release Prep: Branch & Version Bump" | |||
| name: "1. Release branch manual preparation" | |||
There was a problem hiding this comment.
А что за цифры, зачем они нужны?
There was a problem hiding this comment.
И названия скриптов надо оставить один в один как в других репозиториях. Чтобы легко контекст можно было переключать
There was a problem hiding this comment.
Продолжительная идея в том, чтобы при пине в Github Actions этот воркфлоу был наверху, так как это первый шаг
There was a problem hiding this comment.
Тут обсудили на дэйли, хотим попробовать такой формат в KMP, если приживется поправим везде. Дэдлайн пару месяцев
| inputs: | ||
| release_version: | ||
| description: 'Release version (e.g. 1.2.3 or 1.2.3-rc)' | ||
| description: 'Release version (e.g. 1.2.3)' |
There was a problem hiding this comment.
Описания и плашки всё точно также должны быть один в один как в других репозиториях
There was a problem hiding this comment.
Зачем, если KMP у нас не будет под RC версиями?
There was a problem hiding this comment.
а точно не будет. Вот мы катим RC версию. Какая у него должна быть версия common? Без RC, а вдуг изменения в common потребуются?
There was a problem hiding this comment.
Тут обсудили, возвращаем текст про RC
| - name: Merge Pull Request | ||
| run: | | ||
| pr_number=$(gh pr list --base develop --head master --json number --jq '.[0].number') | ||
| gh pr merge $pr_number --merge --auto |
There was a problem hiding this comment.
Если конфиликтов не будет и / или required checks выполнятся успешно - смерджится. Если что-то пойдет не так из этого, просто не произойдет сам merge и останется висеть открыть PR
| gh pr merge $pr_number --merge --auto | ||
|
|
||
| message-to-loop-if-success: | ||
| needs: [merge] |
There was a problem hiding this comment.
Сообщенько в лупу лучше не завязывать на merge, а на пораньше событие по идее. Merge на успешность релиза не влияет.
There was a problem hiding this comment.
Я бы в принципе отсюда убрал шаг с сообщением в Лупу, так как это не реальный релиз, а просто подготовительный этап
There was a problem hiding this comment.
Тут обсудили, удалим сообщение отсюда, это не релиз.
3 participants
WMSDK-547: [CI/CD] Автоматизировать релиз iOS с mindbox-common библиотекой