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

docs: add not about vue-tsc #26076

Closed
wants to merge 1 commit into from
Closed

docs: add not about vue-tsc #26076

wants to merge 1 commit into from

Conversation

cljamal
Copy link
Contributor

@cljamal cljamal commented Mar 4, 2024

Added a note about vue-tsc, since nuxt3 doesn't work with vue-tsc version 2, I lost a lot of time searching for a solution, I hope this little note helps others and saves a lot of time.

about vue-tsc dependency note
Copy link

stackblitz bot commented Mar 4, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

nuxt-studio bot commented Mar 4, 2024

Live Preview ready!

Name Edit Preview Latest Commit
Nuxt Docs Edit on Studio ↗︎ View Live Preview 3f7f0c6

@danielroe
Copy link
Member

danielroe commented Mar 5, 2024

Thank you!

I think there are a few bugs in the latest vue-tsc but it is working in this monorepo (we've updated and run tests with it).

Any issues should probably be raised in http://github.com/vuejs/language-tools - they may already have been resolved.

@danielroe danielroe closed this Mar 5, 2024
@cljamal
Copy link
Contributor Author

cljamal commented Mar 5, 2024

@danielroe Hi!
It's not work in fresh install when you enable typecheck in nuxt.config.ts: https://stackblitz.com/edit/nuxt-starter-pgvktu?file=package.json

@mseeley
Copy link

mseeley commented Mar 13, 2024

Agreed, the works for me attitude is frustrating. It's definitively not working for Nuxt users who refer to Nuxt's documentation.

At least update the docs that "Hey, everyone of you that enables typeCheck: true will run into this bug. We know, it sucks, but it's not us. Please see this link for more information."

@danielroe
Copy link
Member

We did add a note to the docs. See the PR linked at #26083

@mseeley
Copy link

mseeley commented Mar 14, 2024

Timing is impeccable. Thank you @danielroe. I just noticed the same and came back here to correct my comment. 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants