-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Updating for v5 #26
Comments
My personal opinion is to remove the |
If it weren't because the warning log gets missed quite easily when installing, I'd say it provides good feedback to the user. But as it is not forced, and anyway |
Updated the PR, but I think I'm going to leave it as a draft until the workflow v4 tag is available unless there's a reason to merge it in sooner. (Going to follow the same pattern for my other PRs of just leaving them as a draft too.) |
I just saw the issue, sorry for taking over! There are many other repositories available to do an upgrade, so it would still be amazing if you could help! |
Prerequisites
Issue
I'm looking to update this repo for v5. fastify/fastify#5116
Changes Needed (Please let me know if anything might be missing)
UpdateRemove engines from package.jsonengines
in package.json to >= 18The text was updated successfully, but these errors were encountered: