Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: use config sent by pino to support custom message key and log levels #165

Merged
merged 1 commit into from
Apr 26, 2024
Merged

feat: use config sent by pino to support custom message key and log levels #165

merged 1 commit into from
Apr 26, 2024

Conversation

10xLaCroixDrinker
Copy link
Contributor

This implements the new pino feature from pinojs/pino#1925 released in pino-abstract-transport@1.2.0 and pino@8.21.0

Supersedes #155
Fixes #153

BREAKING CHANGE:

  • Requires peerDependency pino ^8.21.0 or ^9.0.0
  • Removes messageKey option
  • Changes severity text to log level name instead of OTel short name

…evels

Supersedes #155
Fixes #153

BREAKING CHANGE:
- Requires pino@^8.21.0 or pino@^9.0.0
- Removes messageKey option
- Changes severity text to log level name instead of OTel short name
@Vunovati
Copy link
Collaborator

This is great 👍🏻

Thanks for the coverage update 🥂

@Vunovati Vunovati merged commit 2bf9bc8 into pinojs:main Apr 26, 2024
7 checks passed
@10xLaCroixDrinker 10xLaCroixDrinker deleted the feature/use-pino-config branch April 26, 2024 20:48
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.

BUG: Short names as SeverityText
2 participants