Skip to content
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 components not refreshing during hmr #305

Closed
wants to merge 14 commits into from

Conversation

PatrykWalach
Copy link
Collaborator

Fixes HMR by method described in #264
Follow up to #264

rbalicki2 and others added 13 commits January 4, 2025 01:44
…ePathToSourceFile

- These aren't relative, yet! But they will be after a future commit.
Note that this currently doesn't match the description. But this is in the interest of breaking up the commit into smaller chunks. Future commits will align the behavior and the description.
- Follows the logic of the AbsoluteEmbeddedLocation struct, namely that it will require an absolute path to Isograph config's project root for std::fmt::Display
@PatrykWalach
Copy link
Collaborator Author

To fix hmr for no babel user we don't have to use WeakMap we have to store resolver next to the wrapped component so we can compare if resolver has changed.

@PatrykWalach PatrykWalach changed the title Fix hmr by changing iso function Fix components not refreshing during hmr Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants