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(vite-vue3): update include regex to support production build #682

Merged
merged 2 commits into from
Aug 21, 2023

Conversation

leedawn
Copy link
Contributor

@leedawn leedawn commented Aug 21, 2023

Description

problem: .The problem just belongs to examples/vite-vue3. There is no component in web page after running 'pnpm build' and 'pnpm preview'

process: After running 'pnpm build', the path '...ComponentA.vue' will become '...ComponentA.vue?vue&type=script&setup=true&lang.ts'. But vite config option 'include' do not include this situation. So I think it's time to change 'include' option's regex. The soluation case is the same as 'core/unplugin.ts'.

Linked Issues

#503

Additional context

Although I soluation the probleam, I don't know the process that the path '...ComponentA.vue' will become '...ComponentA.vue?vue&type=script&setup=true&lang.ts'.

@leedawn leedawn requested a review from antfu as a code owner August 21, 2023 13:03
@antfu antfu changed the title bugfix(vite-vue3): update include regex to support production build fix(vite-vue3): update include regex to support production build Aug 21, 2023
@antfu antfu merged commit 49f8527 into unplugin:main Aug 21, 2023
@leedawn
Copy link
Contributor Author

leedawn commented Aug 21, 2023

Thanks for suggestion and merging.This is my first merged pr. @antfu

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.

None yet

2 participants