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(cli): detect vinxi-based frameworks (@tanstack/start, SolidStart, ...) #6330

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

KitsuneDev
Copy link

This PR adds CLI support for vinxi-based projects, such as @tanstack/start and Solid Start.

Thanks to the fact that those projects are essentially Vite projects, not many changes were necessary, and adding the app.config.* to the detection logic is all that was really necessary. Just to be on the safe side, I also added a simple check that the said file does contain a defineConfig string (and is actually a Vinxi-based config), as the name app.config.* tends to be quite generic.

This has been tested in a barebones @tanstack/start project, and the result can be viewed here. Each commit represents a step of the setup process, following the instructions from the docs.

Copy link

vercel bot commented Jan 11, 2025

@KitsuneDev is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

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