-
-
Notifications
You must be signed in to change notification settings - Fork 20
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: revert node v20 & upgrade deps #213
Conversation
It looks like the changes are going in the reverse of what we want? Can you double check the diff? |
If the goal is to get back to b818e1b, there are still a lot of changes left ( For example, CI is still testing on Node 20. |
I think the problem is that this branch starts from b818e1b. Maybe the following could help: $ git checkout issue212
$ git reset main --hard
$ git revert HEAD~3..HEAD |
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.
LGTM, thanks! Leaving open for @nzakas to verify.
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.
LGTM. 🤞
it reverts the following commits:
it reopens the issue: #205
fixes #212