generated from codemod/migrations-template
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
This codemod recipe is changing logger to loggerInstance, but I believe even in Fastify v5 the normal logger is perfectly valid and still fine to use? https://fastify.dev/docs/latest/Reference/Logging/#enable-logging
The only reason if you want to use loggerInstance is when you want to use a custom logger (which has the same interface as Pino).. But normally you still want to use logger..
Did you know that? If so, maybe that should be documented better.
I also strongly believe this recipe should be removed from https://app.codemod.com/registry/fastify/5/migrationRecipe as well..
Metadata
Metadata
Assignees
Labels
No labels