-
Notifications
You must be signed in to change notification settings - Fork 160
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
UI data-url
in-lining makes builds take forever
#1916
Comments
Changing to this brings the build times down to a reasonable level:
My theory would be that the conversion to a HMR still not working, however. Might be worth checking out: https://stackoverflow.com/questions/71509948/optimizing-images-using-parcel-2 |
One way out of this standoff might be to manage multiple parcelrc files? You're apparently not supposed to do that, but I can't help but think that it would actually solve the problem. Because, the output of But at the same time, But I think we could do {
"transformers:" {
"magic:*": ["...", "@parcel/transformer-raw"]
}
} this is the same transformer as {
"transformers:" {
"magic:*": ["...", "@parcel/transformer-inline-string"]
},
"optimizers": {
"magic:*": ["...", "@parcel/optimizer-data-url"]
}
} that's the same as If we copy the appropriate one into place in the make step, then we could maybe have the behaviour we want in either case? It's absolutely a hack - I feel long term we'd save ourselves pain by using the same toolchain in both projects. But it might help? (definitions of what each URL prefix means: https://github.com/parcel-bundler/parcel/blob/v2/packages/configs/default/index.json ) |
@jpellizzari Is this still relevant/needed/an issue? |
Yes still relevant. This blocks improving our FE build system. |
The
data-url
inlining that was added here has made our builds take a long time and appear to have broken hot-reloading.Reproduce:
ui/lib/images.ts
The text was updated successfully, but these errors were encountered: