Frazer Smith

Want to throw a +1 in for this, in my projects `fastify-swagger` contributes 5-10% to the size of node_modules primarily because of `fastify-static` and the UI (which isn't used as...

> How about version branches? Could add something like `- 'v*'`?

> > Could add something like - 'v*'? > > Yes, we always use that pattern. > > Maybe just `v[0-9.-]{1,8}` if we want to be stricter Doesn't [appear to...

> Just nagging here: What if somebody wants to do a -rc1 suffix? Sorted, just using 'v*' now.

Looks to be related to https://github.com/fastify/fastify-swagger/issues/524

CI failure seems to be unrelated to this PR, could you rerun the workflow when you get a spare second please @mcollina?

Unable to replicate this issue using the following: OS: Windows 10 Home 20H2 Node.js: v14.15.5 pino-pretty: v4.5.0 

Almost there. The only part now throwing errors is: https://github.com/pinojs/pino-elasticsearch/blob/6deb633905a863bcb3d1d71926dea35511ca29ac/lib.js#L62 Which is throwing `this.Connection is not a constructor`. Can't find any reference to changes to this in the v8 release...

> and I notice this project still claims node v10 support: > > > > ``` > > % npm info pino-elasticsearch engines > > { node: '>=10' } >...

> @Fdawgs could you take a look at this? https://github.com/fastify/fastify-elasticsearch was recently updated. Would it just be the CI that needs an update? Not familiar with Elasticsearch at all.