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

feat(create-vite): minimal One.js option to create-vite react templates #18883

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

Conversation

natew
Copy link

@natew natew commented Dec 4, 2024

Description

As discussed with @patak-dev this should add a minimal template for One for easy testing with Vite and for bugs.

One thing I'm not sure of - we expect TARGET_DIR to be just a name like "app-name", we don't accept a path. I can make the changes in the cli easily though to accept a path, but didn't see clearly if this was expected or not here.

@patak-dev patak-dev changed the title Adds minimal One.js option to create-vite react templates feat(create-vite): minimal One.js option to create-vite react templates Dec 4, 2024
@patak-dev
Copy link
Member

TARGET_DIR can be foo/bar so it would be great to add support for that to one. Now that you raise this, I don't know if all the other create-xxx supports this to be honest. The CLI prompt says app name, so I don't think many folks are writing a path there.

About the Minimal template, we don't do this with other frameworks. If there are other templates available, I think you should let the use choose. We only have the barebone requirement for the templates we maintainer in the create-vite package.

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