-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
refactor(angular-query): migrate from tsup to vite #9281
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
refactor(angular-query): migrate from tsup to vite #9281
Conversation
View your CI Pipeline Execution ↗ for commit e6a1444.
☁️ Nx Cloud last updated this comment at |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9281 +/- ##
=======================================
Coverage 45.37% 45.37%
=======================================
Files 207 207
Lines 8276 8276
Branches 1864 1865 +1
=======================================
Hits 3755 3755
Misses 4080 4080
Partials 441 441 🚀 New features to boost your workflow:
|
862240b
to
e6a1444
Compare
@arnoud-dv the production bundle seems to have incorrect |
@hakimio do you mean it should be "types": "./index.d.ts" |
@arnoud-dv no, I mean that |
There's a prepack lifecycle script that makes sure the type declarations are linked from the package root at the moment the package is published. I want to add sub path exports and to support the still widely used |
Ok, let's do it step-by-step:
"exports": {
".": {
"types": "./index.d.ts",
"default": "./dist/index.mjs"
}
} It points to non-existant |
This reverts commit 1c7c276.
I see - I'll revert it for now. Tested this very well, preview package from the PR build was working perfectly on all supported Angular versions and when looking at its files contents it was also good. But probably the actual publish script behaviour is different. I'll look at this later. Thanks for reporting the issue! |
Allows for more flexibility in build process.
Adds prepack and postpack scripts to publish type declaration files in package root, a prerequisite to support subpath exports while also supporting applications using
"moduleResolution": "node"
.Fixes moduleResolution: node16