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

Regression from 2.0.29 to 2.1.4 with @vue-ignore #4842

Open
ig-onoffice-de opened this issue Sep 11, 2024 · 8 comments
Open

Regression from 2.0.29 to 2.1.4 with @vue-ignore #4842

ig-onoffice-de opened this issue Sep 11, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@ig-onoffice-de
Copy link

Vue - Official extension or vue-tsc version

2.1.4

VSCode version

Vue version

3.4.38

TypeScript version

5.5.4

System Info

No response

package.json dependencies

No response

Steps to reproduce

I have a component to wrap another component. The types of v-model do not match (there is currently no other way, already reported the problem here).

I added the @vue-ignore directive a year ago and it was working until now. Now vue-tsc outputs an error.

What is expected?

The @vue-ignore directive should work again.

What is actually happening?

The @vue-ignore directive is ignored.

Link to minimal reproduction

No response

Any additional comments?

No response

@ig-onoffice-de
Copy link
Author

Actually this already happened (see #4232). Maybe there should be better test coverage in this area?

@KazariEX
Copy link
Collaborator

@vue-ignore works fine on my end. Could you provide a minimal reproduction?

@RayGuo-ergou
Copy link
Contributor

RayGuo-ergou commented Sep 11, 2024

image
Cannot reproduce with all same version (vue, vue-tsc and typescript)


without @vue-ignore

image
image

@ig-onoffice-de
Copy link
Author

Minimal Reproduction just here

Just run npm run type-check in the terminal and you will see the error.

@RayGuo-ergou
Copy link
Contributor

It's read only, I cannot run command in terminal. And I also noticed you register a component globally but not declare/import the type. I don't think you would get type mismatch error in this case.

@ig-onoffice-de
Copy link
Author

It's read only, I cannot run command in terminal. And I also noticed you register a component globally but not declare/import the type. I don't think you would get type mismatch error in this case.

Just fork the Example.
I also changed the global to a local import, the same problem.
Actually this code works with 2.0.29.

@RayGuo-ergou
Copy link
Contributor

I see it's not working for props not v-model, for now you can use as any to bypass it.

@KazariEX
Copy link
Collaborator

You can use @vue-expect-error to alleviate this problem.

@KazariEX KazariEX added bug Something isn't working and removed need info labels Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants