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

@ekino/logger v3.0 #48

Open
wants to merge 62 commits into
base: master
Choose a base branch
from
Open

@ekino/logger v3.0 #48

wants to merge 62 commits into from

Conversation

tduyng
Copy link
Contributor

@tduyng tduyng commented Feb 24, 2025

Summary

Rewrite @ekino/logger for v3.0

Motivation

  • It's been so long time the library is not matained
  • Rewrite this library for adapting the modern Node.js (Typescript, ESM)

What has changed?

Here are the key changes made in this PR:

  • Rewritten in ESM
  • Support for dual package (CommonJS and ESM)
  • Rewritten in TypeScript
  • Removed many unnecessary dependencies
  • Replaced yarn with pnpm, which is faster and more convenient with different commands and management
  • Upgraded GitHub Actions, rewrote some deprecated syntax
  • Using git-cliff on GitHub Actions for easily customizable CHANGELOG
  • Replaced ESLint + Prettier with Biome, which is faster, has sufficient rules, and is easier to set up
  • Rewrote all tests from ava + sinon.js + nyc + ts-node to vitest (compatible with ESM)
  • Create a benchmark compare with pino.js and winston
  • Refactored and cleaned up code, focusing on performance, and much more...

Test plan

  • All unit tests have been correctly rewritten with Vitest
  • All functional test behaviors are ensured

tduyng added 30 commits October 21, 2024 16:00
tduyng added 30 commits October 26, 2024 23:50
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.

1 participant