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

Better error message for: Unexpected option config.kit.vite #8207

Closed
LeoDog896 opened this issue Dec 18, 2022 · 2 comments
Closed

Better error message for: Unexpected option config.kit.vite #8207

LeoDog896 opened this issue Dec 18, 2022 · 2 comments

Comments

@LeoDog896
Copy link
Contributor

LeoDog896 commented Dec 18, 2022

Describe the problem

Older blogs use config.kit.vite when creating custom plugins (usually for server hooks), and the removal of this ends up having small reprecussions.

Describe the proposed solution

A small error message that says something similar to "Hint: The Vite config has been moved over to vite.config.??"

Importance

nice to have

@Conduitry
Copy link
Member

A more helpful error message for this, along with many many other things, was removed in #8019. We don't want to be bogged down by these. As is mentioned in the 1.0 blog post (https://svelte.dev/blog/announcing-sveltekit-1.0) and migration guide (#5774) there is a version just before all of these helpful errors were removed that you can upgrade to to help you migrate. I don't think this particular bit of migration deserves special treatment.

@Conduitry Conduitry closed this as not planned Won't fix, can't repro, duplicate, stale Dec 18, 2022
@alexmorrise
Copy link

I appreciate finding this issue because I was following an older blog and couldn't figure out why the plugins were being ignored.

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

No branches or pull requests

3 participants