search

Eomm / fastify-overview

Build a structure graph of your Fastify application
MIT License
65 stars 6 forks source link

feat: add types #61

Closed Eomm closed 1 year ago

Eomm commented 1 year ago

fixes #60

Uzlopak commented 1 year ago

Last nitpick: You use commas in your type definitions. I assume that semicolons are the preferred way and i personally also prefer semicolons.

The typescript documentation uses semicolons everywhere and also a google search says that: https://www.google.com/search?q=typescript+type+comma+or+semicolon&oq=typescript+types+comma+o&aqs=chrome.1.69i57j0i22i30.7144j0j4&client=ms-android-samsung-gj-rev1&sourceid=chrome-mobile&ie=UTF-8

But that is something you can decide.