-
Notifications
You must be signed in to change notification settings - Fork 371
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
fix(module-tools): close declarationMap when bundle dts #4012
Conversation
🦋 Changeset detectedLatest commit: c341efc The changes in this PR will be included in the next version bump. This PR includes changesets to release 216 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4012 +/- ##
==========================================
- Coverage 57.59% 57.30% -0.30%
==========================================
Files 672 645 -27
Lines 17794 17527 -267
Branches 3876 3841 -35
==========================================
- Hits 10249 10044 -205
+ Misses 6934 6859 -75
- Partials 611 624 +13 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at c341efc
This pull request fixes a bug in TypeScript module bundling that caused incorrect declaration maps. It also updates the
@modern-js/module-tools
package version and changelog with a bilingual fix message.Details
🤖 Generated by Copilot at c341efc
@modern-js/module-tools
with a fix message (link)declarationMap: false
fromrollup-plugin-dts
configuration (link)rollup-plugin-dts
that prevents declaration map files from being closed by addingdeclarationMap: false
to the output object (link)Related Issue
Swatinem/rollup-plugin-dts#113
Closes #4009
Checklist
pnpm run change
.