Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: hmr reload should work with async component #11248

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Jul 15, 2024
Merged

fix: hmr reload should work with async component #11248

merged 12 commits into from
Jul 15, 2024

Conversation

@kermanx
Copy link
Contributor

@kermanx kermanx commented Jun 28, 2024 •
edited
Loading

It's now blocking slidevjs/slidev#1708

@kermanx kermanx mentioned this pull request Jun 28, 2024
Merged
@github-actions
Copy link

github-actions bot commented Jun 28, 2024 •
edited
Loading

Size Report

Bundles

File Size Gzip Brotli
runtime-dom.global.prod.js 89.6 kB 34.2 kB 30.9 kB
vue.global.prod.js 146 kB 53.7 kB 47.8 kB

Usages

Name Size Gzip Brotli
createApp 51.5 kB 20.1 kB 18.3 kB
createSSRApp 54.8 kB 21.5 kB 19.6 kB
defineCustomElement 53.8 kB 20.9 kB 19 kB
overall 65.2 kB 25.2 kB 22.7 kB

@antfu antfu added scope: hmr 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. ready for review This PR requires more reviews regression labels Jul 10, 2024
@yyx990803 yyx990803 merged commit c8b9794 into vuejs:main Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@antfu antfu antfu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. ready for review This PR requires more reviews regression scope: hmr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kermanx @antfu @yyx990803