Skip to content

Conversation

@flofehrenbacher-sz
Copy link
Contributor

Thanks for providing the package pino-lambda.
We use it in combination with pino 7 which seems to work just fine, only npm is making troubles because of the peerDependency pino declared as ^6.0.0. Would you consider to change it to >=6.0.0?

This might resolve #26

We know it works with pino version 7 but we don't know yet about further upgrades
@flofehrenbacher-sz
Copy link
Contributor Author

To be more strict I changed it to >=6.0.0 <8.0.0 because we can't know if it will work with a further breaking change update from pino.

@carbonrobot
Copy link
Contributor

Thanks for the PR @flofehrenbacher-sz! Will merge and release shortly.

@carbonrobot carbonrobot merged commit a9cb3a4 into FormidableLabs:master Apr 9, 2022
github-actions bot pushed a commit that referenced this pull request Apr 9, 2022
### [3.0.1](v3.0.0...v3.0.1) (2022-04-09)

### Bug Fixes

* allow higher pino versions ([#30](#30)) ([a9cb3a4](a9cb3a4))
@github-actions
Copy link

github-actions bot commented Apr 9, 2022

🎉 This PR is included in version 3.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Outdated pino peer dependency

2 participants