-
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
Rename Module #78
Rename Module #78
Conversation
I got a 128 status code when running it through So, I've created this PR manually. |
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
Could you update the readme with the new name, and the plugin name as well please @RafaelGSS? |
The plugin name should remain the same for |
Good point |
@RafaelGSS You can pull latest main to fix linting |
This commit renames the module in accordance with the discussion in fastify/fastify#3733
Can someone merge it? I don't have permission in this repo. |
This pull request renames the module according to the discussion in
fastify/fastify#3733.
Note that the deprecation module has already been published and that the
code for it does not exist in this repository.
Coordinating the drastic changes to the code for the module deprecation and
then restoring the code for the module renaming would have been extremely
difficult and prohibitively tedious.
Important: no further releases should be added to the old major version.