-
-
Notifications
You must be signed in to change notification settings - Fork 34.5k
build,deps: replace cjs-module-lexer with merve #61456
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
base: main
Are you sure you want to change the base?
Conversation
|
Review requested:
|
ae6ab98 to
5e69495
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #61456 +/- ##
========================================
Coverage 89.85% 89.86%
========================================
Files 671 672 +1
Lines 203166 203322 +156
Branches 39057 39085 +28
========================================
+ Hits 182554 182713 +159
+ Misses 12967 12940 -27
- Partials 7645 7669 +24
🚀 New features to boost your workflow:
|
9bfb358 to
1da16dd
Compare
66e99b4 to
d687c0a
Compare
|
Where does this new dep live? |
Currently, under my github account. Is anybody interested in helping me maintain it? github.com/anonrig/merve |
d687c0a to
b2ec704
Compare
|
Per current process it would make sense to move merve into the Node.js org as well. I can then add a note to the cjs-module-lexer readme that future bugs should be posted there. I'd value also having permissions to the new repo to continue to provide support. |
lemire
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Next step, name a library after your cat.
Initially started a couple of years ago, the new cjs module lexer is 25% faster for cold importing cjs files. This change introduces a new dependency and removes the old one.
Benchmark CI: https://ci.nodejs.org/view/Node.js%20benchmark/job/benchmark-node-micro-benchmarks/1784/console