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(pino): support new pino version ^9.0.0 #2163

Merged
merged 8 commits into from Apr 30, 2024

Conversation

blumamir
Copy link
Member

fix #2155

This PR is to add support for new pino major version 9 released on 24 April 2024 (5 days ago).

According to release notes, the only change is "Dropped support for Node.js 14, 16, & 19".

Added version 9 to supported versions, updated tav.yaml configuration and the README. run test-all-versions locally to verify

Copy link

codecov bot commented Apr 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.12%. Comparing base (dfb2dff) to head (f12e4a2).
Report is 96 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2163      +/-   ##
==========================================
+ Coverage   90.97%   95.12%   +4.14%     
==========================================
  Files         146       10     -136     
  Lines        7492      697    -6795     
  Branches     1502      142    -1360     
==========================================
- Hits         6816      663    -6153     
+ Misses        676       34     -642     

see 137 files with indirect coverage changes

@pichlermarc pichlermarc merged commit 585b833 into open-telemetry:main Apr 30, 2024
17 checks passed
@dyladan dyladan mentioned this pull request Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

opentelemetry-instrumentation-pino not working with pino 9.0.0
4 participants