Skip to content
New issue

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

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

Already on GitHub? # to your account

v0.4.0 #333

Merged
merged 13 commits into from
Apr 5, 2023
Merged

v0.4.0 #333

merged 13 commits into from
Apr 5, 2023

Conversation

stuft2
Copy link
Contributor

@stuft2 stuft2 commented Apr 5, 2023

  • BREAKING CHANGE: Drop node 12 support
  • BREAKING CHANGE: Drop support for Pino v6
  • chore(deps): update dependencies
  • chore(license): update year
  • feat: export ByuLogger
  • fix: improve type check
  • style: lint test

@stuft2 stuft2 requested a review from a team April 5, 2023 06:23
@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Merging #333 (d3f4b40) into main (e81428a) will increase coverage by 2.72%.
The diff coverage is 90.90%.

❗ Current head d3f4b40 differs from pull request most recent head 3f1b5f1. Consider uploading reports for the commit 3f1b5f1 to get more accurate results

@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
+ Coverage   77.27%   80.00%   +2.72%     
==========================================
  Files           2        2              
  Lines          22       25       +3     
  Branches        6        6              
==========================================
+ Hits           17       20       +3     
  Misses          5        5              
Impacted Files Coverage Δ
src/util.ts 66.66% <66.66%> (+5.12%) ⬆️
src/logger.ts 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@stuft2 stuft2 changed the title v0.4.1 v0.5.0 Apr 5, 2023
@@ -1,9 +1,9 @@
import Pino from 'pino'
import { getLevel, isInstalled } from './util'
import { Logger, LoggerOptions, pino } from 'pino'
Copy link
Contributor Author

@stuft2 stuft2 Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is preferred over default exports. I was seeing the same error as in this issue when trying to import pino and our logger in an ES Module: pinojs/pino#1452 (comment)

@stuft2 stuft2 changed the title v0.5.0 v0.4.0 Apr 5, 2023
@stuft2 stuft2 merged commit d2f92f8 into main Apr 5, 2023
@stuft2 stuft2 deleted the patch/deps branch April 5, 2023 16:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants