search

hapijs / hapi-pino

🌲 Hapi plugin for the Pino logger
MIT License
148 stars 61 forks source link

Bumped v8.4.0 #139

Closed lauraseidler closed 3 years ago

lauraseidler commented 3 years ago

Hey,

any chance to get the changes from https://github.com/pinojs/hapi-pino/pull/138 out as a release? I've installed from GitHub by commit hash until now, but it's now making some problems, so I'd love to have an npm release :)

Thanks!

mcollina commented 3 years ago

I've added you to npm so you can publish yourself.

mcollina commented 3 years ago

Sorry about this, I'm not using this anymore and doing a bad job at releasing it! I hope you don't mind.

lauraseidler commented 3 years ago

@mcollina no problem at all, thanks! Anything in particular I need to be aware of?

mcollina commented 3 years ago

nope ;), just go ahead and ship it. We usually push version bumps straight to master at publish time, no pr.

lauraseidler commented 3 years ago

Cheers! I will merge this one as it's already there, will push directly for the next one :)

Btw, I see that travis builds aren't running anymore (travis-ci.org not a thing anymore) - given that the build is not super complicated, I thought instead of moving to travis-ci.com, GitHub actions might be an option? WDYT?

mcollina commented 3 years ago

go for actions!

mcollina commented 3 years ago

(also add yourself as a contributor to package.json).