-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix: update to node 18.18.2 and node 16.20.2 #4
Conversation
@n0cloud Thanks for your PR! Sorry for the delay but I missed it at start, the expected sha must be updated by me once I create a new patch release so don't worry about them :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@n0cloud Could you try to run pkg-fetch locally using -t
flag?
Check https://github.com/yao-pkg/pkg-fetch/blob/main/DEVELOPMENT.md for more info
Also not sure but maybe latest node 18 is 18.18.2 now?
I'm getting this error when I test, but v19 also throw this error. |
@baparham any clue where that error comes from? |
@n0cloud The verify script runs this code: https://github.com/yao-pkg/pkg-fetch/blob/main/lib/verify.ts#L6-L94 I think one of those assertions throws, could you edit it manually to see which one? |
Th test script check for My latest commit should fix it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @n0cloud! If you confirm me that running pkg-fetch with -t
option works I will merge this 🎉
I confirm |
Let's see how this goes: https://github.com/yao-pkg/pkg-fetch/actions/runs/6536202980 |
@n0cloud Are you someway able to create node 20 patches? |
PKG 5.9.2 published https://github.com/yao-pkg/pkg/releases/tag/v5.9.2 🎉 |
will need to update expected-shas.json, but I don't remember how to do that.