fix: fix outdated type definition in ValidationResult interface

[micheleriva]

Hi there,
this commit fixes #4045. There was an outdated type reference inside of fastify.d.ts that caused a type mismatch with the AJV library.

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows the Developer's Certification of Origin
  and the Code of conduct

[mcollina]

Thanks for opening a PR! Can you please add a unit test? We use tsd to test our types

[RafaelGSS]

Thanks for opening a PR! Can you please add a unit test? We use tsd to test our types

Looks like we don't maintain tests for ValidationResult so far. It would be a great addition.

Also, it would need to be updated as well: https://github.com/fastify/fastify/blob/9e21d2829ad611dd57aec22eccb202ed39cf4f6f/docs/Reference/TypeScript.md#fastifyvalidationresult.

[mcollina]

why did you close the PR?

[micheleriva]

@mcollina I've seen that there's another PR fixing this problem and wanted to avoid duplications 🙂

[mcollina]

which one?

[RafaelGSS]

which one?

#4070

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.