add symbol type to decorators

J-Hoh commented 2 years ago

fastify decorators can be either of type string or symbol, the later was missing from the typings

Checklist

[X] run npm run test and npm run benchmark (benchmark script isnt present?)
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[X] commit message and code follows the Developer's Certification of Origin and the Code of conduct

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1741081073

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 1595860612:	0.0%
Covered Lines:	0
Relevant Lines:	0

💛 - Coveralls

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1737753668

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 1595860612:	0.0%
Covered Lines:	40
Relevant Lines:	40

💛 - Coveralls

J-Hoh commented 2 years ago

Didnt even know about tsd, added a symbol type to both decorator type checks. Hope the force push was okay

fastify / fastify-plugin