refactor: prefix unused params with underscores (#162)

fastify · Jan 5, 2025 · d2bc2d4 · d2bc2d4
1 parent 21bb215
commit d2bc2d4
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -3,8 +3,8 @@
 const isPromise = require('node:util').types.isPromise
 const fp = require('fastify-plugin')
 
-function fastifyFunky (fastify, opts, next) {
-  fastify.addHook('preSerialization', (req, res, payload, done) => {
+function fastifyFunky (fastify, _opts, next) {
+  fastify.addHook('preSerialization', (_req, res, payload, done) => {
     // Handle Either
     if (isEither(payload)) {
       return resolvePayload(done, payload.left, payload.right, res)

diff --git a/test/helpers.js b/test/helpers.js
@@ -9,7 +9,7 @@ function initAppGet (t, endpoint) {
 
   app.get('/', endpoint)
 
-  app.setErrorHandler((error, request, reply) => {
+  app.setErrorHandler((error, _request, reply) => {
     app.log.error(error)
     t.assert.deepStrictEqual(error.message, 'Invalid state')
     reply.status(500).send({ ok: false })

diff --git a/types/index.test-d.ts b/types/index.test-d.ts
@@ -26,7 +26,7 @@ expectType<any>(fastifyFunkyCjs)
 
 app.register(fastifyFunkyDefault)
 // this gives a type error:
-app.get('/', (req: FastifyRequest, reply: FastifyReply) => {
+app.get('/', (_req: FastifyRequest, _reply: FastifyReply) => {
   return { right: { id: 1 } }
 })