-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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: do not cache module while the file contains import.meta.glob #3005
Conversation
tests are not successful |
@Niputi This error does not seem to be caused by my submission |
could you try rebasing against main? |
@Niputi done |
Personally don't think this is the best way to solve the problem. Introduce fs operations and disable the cache could have certain impact to the pref. Better to track the dependencies and invalid the module on deps changes properly instead. |
@antfu Thanks for your suggestion. I re-examined the source code and found that this problem was caused by the invalidate method does not consider the ssr module. I revised it again, please take a look. Thank you very much! |
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.
This change works well for my use cases!
Thank you. Let's hope it will be merged soon.
…tejs#3005) Co-authored-by: ygj6 <ygj@54321>
Description
in #2734 , while source file(
app.js
) useimport.glob.meta
to scan/src/data/*.json
,while we add a json file such asx.json
,the module will hit cache in urlToModuleMap because the source file(app.js
) did not change.fix #2734
Additional context