search

dbhowell / pino-cloudwatch

AWS CloudWatch Logs transport for pino
MIT License
38 stars 13 forks source link

Replace lodash.reduce with Array.prototype.reduce #24

Closed wojtekmaj closed 4 years ago

wojtekmaj commented 4 years ago

lodash.reduce worked by pure luck (!) that this dependency is used by another dependency.

So there are two options to resolve this issue: a) Add it to dependencies b) Remove its usage

wojtekmaj commented 4 years ago

Passed tests locally on Node v12.13.0, macOS Catalina.

dbhowell commented 4 years ago

Hey @wojtekmaj thanks for picking this up! Good catch.

Best to remove it as we don't need it now like you've suggested.