feat: add assets inline exclude#6892
feat: add assets inline exclude#6892web-97 wants to merge 3 commits intovitejs:mainfrom web-97:feat/add-assetsInlineExclude
Conversation
|
Closes #2173 |
|
Thanks for the PR! I think that it would be good to refactor the options to be So we can use the usual include/exclude But let's wait to see what other maintainers think. I'll add it for discussion in our next team meeting (we just had one, so a bit of patience since we are going to get to this one in two weeks). |
Thanks, looking forward to the next changes |
|
Any update on this? would love this feature 🙏 |
|
emmm... I also think the PR need to #6892 (comment). |
|
I also have this problem, but I don't think this is a correct solution. May I offer an alternative solution Make it a callback with the file, filesize, and just for kicks, current total size of everything included so far. |
|
Cleaning up stale PRs. The discussion notes for this favoured the API in #8717 instead with a function form. Closing this for now. |
Description
Closes #2173
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123).