Исправить воркфлоу pr-preview.yml

StarHamster commented 2 years ago

Что случилось

Я не очень разбираюсь в workflows, но тут явно что-то не так. Скрипт два раза пытается достучаться до переменной steps.links.outputs.list, но у него ничего не выходит.

Вот скрипт стучится в первый раз:

https://github.com/doka-guide/content/blob/dc414110a0a8ae18c6de1d60b799762736e3f277/.github/workflows/pr-preview.yml#L100

Фактически он должен в логах что-то вывести, но этого не происходит и выводится пустая строка:
Когда скрипт стучится во второй раз, содержимое этой переменной должно быть в <details>. Но этого не происходит, поэтому <details> здесь получается лишним.

https://github.com/doka-guide/content/blob/dc414110a0a8ae18c6de1d60b799762736e3f277/.github/workflows/pr-preview.yml#L120

Что делать

Я вижу два варианта, что можно сделать с этим:

Понять, какая правильная переменная
Удалить первую строчку полностью, а во второй убрать <details> и эту самую переменную, чтобы осталось вот так:
```
message: '<a href="${{ env.DEPLOY_DOMAIN }}">Превью контента</a> из ${{ github.event.after }} опубликовано.'
```

P.S. У меня вопрос: а вообще почему это флоу так ошибками сыплет?

igsekor commented 2 years ago

Спасибо за то, что заметили. Мы сейчас переделываем этот экшен, поэтому не всё ещё работает корректно. Есть артефакты из прошлых версий. В ближайшее время поправим 😇

Dfhjbcccc commented 2 years ago

Что случилось

Я не очень разбираюсь в workflows, но тут явно что-то не так. Скрипт два раза пытается достучаться до переменной steps.links.outputs.list, но у него ничего не выходит.

Вот скрипт стучится в первый раз:

https://github.com/doka-guide/content/blob/dc414110a0a8ae18c6de1d60b799762736e3f277/.github/workflows/pr-preview.yml#L100

Фактически он должен в логах что-то вывести, но этого не происходит и выводится пустая строка:

Когда скрипт стучится во второй раз, содержимое этой переменной должно быть в <details>. Но этого не происходит, поэтому <details> здесь получается лишним.

https://github.com/doka-guide/content/blob/dc414110a0a8ae18c6de1d60b799762736e3f277/.github/workflows/pr-preview.yml#L120

Что делать

Я вижу два варианта, что можно сделать с этим:
Понять, какая правильная переменная
Удалить первую строчку полностью, а во второй убрать <details> и эту самую переменную, чтобы осталось вот так:
message: '<a href="${{ env.DEPLOY_DOMAIN }}">Превью контента</a> из ${{ github.event.after }} опубликовано.'
P.S. У меня вопрос: а вообще почему это флоу так ошибками сыплет?

doka-guide / content