Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Почему не работает на 18 nodejs? #106

Closed
omqmo opened this issue Aug 27, 2023 · 3 comments · Fixed by #110
Closed

Почему не работает на 18 nodejs? #106

omqmo opened this issue Aug 27, 2023 · 3 comments · Fixed by #110

Comments

@omqmo
Copy link

omqmo commented Aug 27, 2023

Как запустить serverless-yandex-cloud на nodejs18+? Не знаю, как в глобальных масштабах, но на github-actions из-за указанного пакета зависимости необходимо не устанавливать, а пересобирать с помощью команды npm rebuild. Это, как минимум, ломает nx. Разбираться в совместимости с устаревшей версией nodejs для каждой из сотен пакетов nx,ну вообще сами понимаете никто не будет. Поэтому, вопрос - в чем проблема переписать упомянутый пакет на актуальные версии ноды?

Если у кого-то есть рабочий пример github-actions + serverless + nx буду очень рад, если им поделитесь.

@nikolaymatrosov
Copy link
Contributor

А зачем вы используете этот пакет в github actions? Есть же отдельный экшен для деплоя функции https://github.com/yc-actions/yc-sls-function

@omqmo
Copy link
Author

omqmo commented Aug 27, 2023

@nikolaymatrosov оу, мне этот не попадался. Спасибо, сейчас попробую.

А зачем вы используете этот пакет в github actions?

action для нативного serverless, который также подключает плагины. Лучше я ничего не смог найти, но ваш вариант выглядит аналогичным.

@DavyJohnes
Copy link
Contributor

из-за указанного пакета зависимости необходимо не устанавливать, а пересобирать с помощью команды npm rebuild

Можно тут подробнее, пожалуйста? Помечу нельзя просто установить плагин как зависимость и использовать его, как указано в документации?

@DavyJohnes DavyJohnes linked a pull request Dec 11, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants