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: better bundling of types #1785
fix: better bundling of types #1785
Conversation
Looking for a way to fully ignore |
I found a wonderful flag |
@@ -25,6 +25,7 @@ | |||
"test:ci": "cross-env CI=true pnpm -r --stream --filter !test-fails --filter !test-browser run test --allowOnly", | |||
"test:ci:single-thread": "cross-env CI=true pnpm -r --stream --filter !test-fails run test --allowOnly --no-threads", | |||
"typecheck": "tsc --noEmit", | |||
"typecheck:why": "tsc --noEmit --explainFiles > explainTypes.txt", |
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.
👀? Cool magic good to know!
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.
Wanted to keep it in case someone will mess up the types again 👀
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.
I don't completely understand what is happening but I trust your judge :P
Reduces the bundle size (and time for
build
), since we no longer bundle the sameVitest
type (and its dependencies).Before:
After:
Possible fix for #1619?