-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Update for Fastify 2.0 #96
Comments
Thanks, would you like to send a PR? |
@mcollina Should we update |
what do you mean? I would just release a new major for this, or relax https://github.com/fastify/point-of-view/blob/master/index.js#L288. |
I meant whether it is alright to relax the constraint without bumping https://github.com/fastify/point-of-view/blob/master/package.json#L43 |
if it works, I have no problem in that
Il giorno mer 23 gen 2019 alle ore 11:26 Cemre Mengu <
notifications@github.com> ha scritto:
… I meant whether it is alright to relax the constraint without bumping
https://github.com/fastify/point-of-view/blob/master/package.json#L43
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#96 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADL49cl7A0k1he_gS0h5E0-6892ST2hks5vGEbegaJpZM4aNtKl>
.
|
Sorry @mcollina I would love to but I'm too busy right now. I opened the issue in case someone had more time than me and to make sure it would eventually be done. |
Landed in #99 |
Error: fastify-plugin - expected '^1.1.0' fastify version, '2.0.0' is installed
If/when someone have time,
😄 🐭 🧀
The text was updated successfully, but these errors were encountered: