Skip to content

feat(deps): move pino to peerDependencies#42

Merged
KodrAus merged 1 commit intodatalust:devfrom
volebo:dev
May 18, 2025
Merged

feat(deps): move pino to peerDependencies#42
KodrAus merged 1 commit intodatalust:devfrom
volebo:dev

Conversation

@maxkoryukov
Copy link
Copy Markdown
Contributor

Hello!

I tried to follow guides and articles from NodeJS team and contributors and moved the pino dependency to the peerDependencies section in package.json

@KodrAus
Copy link
Copy Markdown
Member

KodrAus commented May 18, 2025

Thanks for working on this @maxkoryukov! This looks good to me. I'm not sure if it's technically a breaking change, but since pino-seq doesn't work without pino anyways I think this is ok for us to release in a minor version.

@KodrAus KodrAus merged commit 601a018 into datalust:dev May 18, 2025
9 checks passed
This was referenced May 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants