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(language-core): fallthrough attributes break component type when root tag type is unknown #4729

Merged
merged 2 commits into from
Aug 27, 2024

Conversation

johnsoncodehk
Copy link
Member

@johnsoncodehk johnsoncodehk commented Aug 27, 2024

When the root tag type is unknown and the component missing props define, the component type is corrupted. This is actually defineComponent's bug and trying to avoid it on our side.

Related: vuejs/core#7419

Copy link

pkg-pr-new bot commented Aug 27, 2024

commit: 30f7455

vue-component-meta

pnpm add https://pkg.pr.new/vuejs/language-tools/vue-component-meta@4729

vue-component-type-helpers

pnpm add https://pkg.pr.new/vuejs/language-tools/vue-component-type-helpers@4729

@vue/language-core

pnpm add https://pkg.pr.new/vuejs/language-tools/@vue/language-core@4729

@vue/language-plugin-pug

pnpm add https://pkg.pr.new/vuejs/language-tools/@vue/language-plugin-pug@4729

@vue/language-server

pnpm add https://pkg.pr.new/vuejs/language-tools/@vue/language-server@4729

@vue/language-service

pnpm add https://pkg.pr.new/vuejs/language-tools/@vue/language-service@4729

vue-tsc

pnpm add https://pkg.pr.new/vuejs/language-tools/vue-tsc@4729

@vue/typescript-plugin

pnpm add https://pkg.pr.new/vuejs/language-tools/@vue/typescript-plugin@4729

Open in Stackblitz

@johnsoncodehk johnsoncodehk marked this pull request as ready for review August 27, 2024 09:58
@johnsoncodehk johnsoncodehk merged commit 51bd930 into master Aug 27, 2024
9 checks passed
@johnsoncodehk johnsoncodehk deleted the inheritAttrsForUnknownTag branch August 27, 2024 10:02
johnsoncodehk added a commit that referenced this pull request Aug 27, 2024
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.

1 participant