plasma-infra: Upload assets when publish rc

[Yakutoc]

Upload assets when publish RC

• переопределена логика получения commit для правильного условия загрузки assets в момент публикации RC

What/why changed

Когда мы находимся в dev ветки команда - npx lerna la --json --since=$(git merge-base --fork-point origin/dev) не видит изменений.

Поэтому для publish RC решили изменить подход отслеживания изменений с момента последнего выпуска.

Lerna changed ➖

В таком подходе есть уязвимость, например

В прошлом у нас были внесены изменения в @salutejs/plasma-tokens-native но с префиксом chore и соотвественно такие изменения не сделали bump версии пакета и не было выпуска, это ключевой момент.

И поэтому вывод команды npx lerna changed --all --json учтет эти изменения, они не были опубликованы.

И расчет условия загрузки для ваших изменений будет некорректным, потому что вывод npx lerna changed содержит запись о изменениях в plasma-tokens.

Получить корректный ref, sha для --since ✅

В момент выпуска RC у нас есть commit который запустил процесс публикации.

На основе этого commit можно получить список связанных с ним pull requests (API listPullRequestsAssociatedWithCommit).

Из полученных данных вытаскиваем last commit (base.sha) для ветки dev на момент влития pull request.

Теперь lerna корректно может сравнить состояние изменений между внесенными и тем что уже есть в dev.

Lerna ls --since with correct sha

last commit (base.sha) до влития изменений

пример из песочницы

[Salute-Eva]

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-892/

[Yeti-or]

Я не понял почему мы считаем что npx lerna changed --all --json пишет некоректный вывод?
ведь если там chore то публиковать ничего не надо в том числе assets

[Yeti-or]

По изменениям все ок но я не понял зачем нужно так сложно из твоего примера в описании мне не ясно почему это нужно делать так

[Yakutoc]

Я не понял почему мы считаем что npx lerna changed --all --json пишет некоректный вывод? ведь если там chore то публиковать ничего не надо в том числе assets

@Yeti-or

например твои изменения не затрагивали ничего в tokens и ты ожидаешь что загрузки assets не будет.

Однако npx lerna changed видит неопубликованные изменения в tokens, что были сделаны раннее, и при публикации твоих изменений начинает грузить assets, потому что вывод npx lerna changed содержит запись о plasma-tokens.

Так стало понятнее?

[Yeti-or]

Да теперь понял

[Salute-Eva]

🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀