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

Add Node 20 #9

Closed
segevfiner opened this issue Oct 25, 2023 · 23 comments
Closed

Add Node 20 #9

segevfiner opened this issue Oct 25, 2023 · 23 comments

Comments

@segevfiner
Copy link

It's now the latest LTS.

@robertsLando
Copy link
Member

Unfortunately we don't have anyone right now able to create the v20 patches

@glektarssza
Copy link

Would be addressed by #11

@robertsLando
Copy link
Member

Fixed in #11

@viceice
Copy link

viceice commented Dec 4, 2023

i think this should be re-opened until the build issues are fix.

@robertsLando robertsLando reopened this Dec 4, 2023
@robertsLando
Copy link
Member

@robertsLando
Copy link
Member

Seems the build is going some steps further this time: https://github.com/yao-pkg/pkg-fetch/actions/runs/7087579394/job/19288161530 🎉

@viceice
Copy link

viceice commented Dec 4, 2023

nice, it seems you should update some actions which will otherwise fail in near future

Warning: The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

it's the docker build and push action.

@glektarssza
Copy link

nice, it seems you should update some actions which will otherwise fail in near future

Warning: The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

it's the docker build and push action.

@viceice I can open a separate ticket and take a look at this in the next few days if you want.

@viceice
Copy link

viceice commented Dec 4, 2023

nice, it seems you should update some actions which will otherwise fail in near future

Warning: The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

it's the docker build and push action.

@viceice I can open a separate ticket and take a look at this in the next few days if you want.

I would suggest to use renovate for automated updates (I'm a maintainer 😉)

@glektarssza
Copy link

nice, it seems you should update some actions which will otherwise fail in near future

Warning: The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

it's the docker build and push action.

@viceice I can open a separate ticket and take a look at this in the next few days if you want.

I would suggest to use renovate for automated updates (I'm a maintainer 😉)

@viceice Sure, I can look at it! I'll open an issue to track.

@robertsLando
Copy link
Member

robertsLando commented Dec 5, 2023

Seems some matrix was missing :( need to trigger another build

28d96e9

Closing this

@robertsLando
Copy link
Member

will remove nodejs19 on next release, just because we need to be sure v20 works propely before dropping it. Every feedback is welcome!

@viceice
Copy link

viceice commented Dec 5, 2023

Will try v20 at https://github.com/containerbase/base now

@viceice
Copy link

viceice commented Dec 5, 2023

@robertsLando Can you publish a new release of @yao-pkg/pkg-fetch and @yao-pkg/pkg to include the v20?

Error [ReportedError]: No available node version satisfies 'v20'

🙃

@robertsLando
Copy link
Member

@viceice Actually I'm waiting for this to end: https://github.com/yao-pkg/pkg-fetch/actions/runs/7097555049

linux-arm64 and macos arm64 binaries were missing.

Anyway I suggest you to clone both packages locally and use npm link

@viceice
Copy link

viceice commented Dec 5, 2023

@viceice Actually I'm waiting for this to end: https://github.com/yao-pkg/pkg-fetch/actions/runs/7097555049

linux-arm64 and macos arm64 binaries were missing.

All done 🙃

Anyway I suggest you to clone both packages locally and use npm link

Will try, but not enough time for that yet 😉

@robertsLando
Copy link
Member

robertsLando commented Dec 5, 2023

pkg 5.11.0 is out now

https://github.com/yao-pkg/pkg/releases/tag/v5.11.0

@viceice
Copy link

viceice commented Dec 5, 2023

will try tomorrow

@glektarssza
Copy link

Seems some matrix was missing :( need to trigger another build

28d96e9

Closing this

How on earth did I miss that... :<

Oh well, glad you caught it.

@robertsLando
Copy link
Member

No worries @glektarssza you already did a lot with that patch 🙏🏻

@viceice
Copy link

viceice commented Dec 6, 2023

Seems to work for amd64 and arm64 🎉

@robertsLando
Copy link
Member

Niceee 🚀

@robertsLando
Copy link
Member

Just created a workflow that automatically bumps patches (at least when them apply cleanly) 🚀

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

No branches or pull requests

4 participants