-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
release: v2.5.1 #84
release: v2.5.1 #84
Conversation
Codecov Report
@@ Coverage Diff @@
## main #84 +/- ##
=======================================
Coverage 99.14% 99.14%
=======================================
Files 7 7
Lines 468 468
Branches 84 84
=======================================
Hits 464 464
Misses 4 4 📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
fix: fix cjs -> esm breakchange, to #82
"exports": { | ||
"import": "./dist/index.mjs", | ||
"require": "./dist/index.cjs" | ||
}, | ||
"main": "dist/index.mjs", |
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.
This is still a breaking change for pre-exports node versions. Your "main" must be a .js
file that's CJS to avoid that.
Changes
Base PullRequest
default branch (https://github.com/technote-space/doctoc/tree/main)
Command results
Details:
add path
cli.js -u --packageFile package.json
yarn install
yarn install v1.22.19 [1/4] Resolving packages... [2/4] Fetching packages... [3/4] Linking dependencies... [4/4] Building fresh packages... success Saved lockfile. Done in 14.97s.
yarn upgrade
yarn audit
yarn audit v1.22.19 0 vulnerabilities found - Packages audited: 512 Done in 0.53s.
Changed files
Changed 2 files: