Update logzio-nodejs dependency

logzio / logzio-docker

Forward all your Docker logs to Logz.io

Apache License 2.0

35 stars 17 forks source link

Update logzio-nodejs dependency #22

Closed avnersorek closed 5 years ago

avnersorek commented 5 years ago

The old version does not allow overiding the type attribute, which we needed to do. Thanks

idohalevi commented 5 years ago

@avnersorek Hi, thank you for the PR. I just need to check that that nothing is going to break since we are bumping the version. Have you seen any issues on your end?

avnersorek commented 5 years ago

@idohalevi , so far so good - no issues. We've been using this version for like 3 days now :|

idohalevi commented 5 years ago

@avnersorek Thank you for the feedback. I'll go over it during the week

idohalevi commented 5 years ago

@avnersorek Sorry for the long wait. Can you just increase the version before I'll merge?

avnersorek commented 5 years ago

Thanks @idohalevi No problem and no hurry. Should the version be bumped to 2.0.1 or 2.1.0 ?

idohalevi commented 5 years ago

@avnersorek 2.0.1 is fine :)

avnersorek commented 5 years ago

@idohalevi Done 👍

avnersorek commented 5 years ago

@adrukh