Skip to content

Document TypeScript issues in app.ts and tsconfig.json (VS Code warnings) #134

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

Closed
wants to merge 1 commit into from

Conversation

mark1502
Copy link

@mark1502 mark1502 commented May 1, 2025

This PR documents two VS Code TypeScript warnings encountered when using the starter kit:

  1. TS2306 in app.ts

    • "File 'vite/client.d.ts' is not a module."
    • This comes from import 'vite/client'; and can be safely ignored.
    • A comment has been added to explain the warning and link to the relevant Vite issue.
  2. Missing type definition 'vue/tsx' in tsconfig.json

    • "Cannot find type definition file for 'vue/tsx'"
    • This likely comes from the line "types": ["vue/tsx"] without the @vue/tsx-jsx package being installed.
    • A comment has been added suggesting removal or further clarification.

These TypeScript warnings may be confusing to new users of the vue-starter-kit.
The additions are mostly intended to bring this to the attention of the developers of the vue-starter-kit and could also help new users who may be confused by the warnings.

@TimKunze96
Copy link

TimKunze96 commented May 2, 2025

Both of these errors would get fixed by #127

The vue/tsx line can safely get removed, and the type error is fixed once the type definition is moved to a .d.ts file

@mark1502
Copy link
Author

mark1502 commented May 2, 2025

Both of these errors would get fixed by #127

The vue/tsx line can safely get removed, and the type error is fixed once the type definition is moved to a .d.ts file

Sounds good, I guess I'll close the issue. Thanks!

@mark1502 mark1502 closed this May 2, 2025
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.

2 participants