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

chore: add create-preact recommendation to Preact templates #17649

Merged
merged 3 commits into from
Jul 14, 2024

Conversation

rschristian
Copy link
Contributor

Description

Like the Vue template does, I think we, Preact, would prefer to nudge users towards our own initializer which offers a bit more Preact-specific functionality and allows us to control the defaults of a bit better.

Copy link

stackblitz bot commented Jul 10, 2024

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

@rschristian rschristian changed the title chore: Add create-preact recommendation to Preact templates chore: add create-preact recommendation to Preact templates Jul 10, 2024
@patak-dev
Copy link
Member

Sounds good! @rschristian, I think we should also add an option in the CLI when the user selects Preact. See how the user can jump to create-vue from create-vite. Let me know if you'd like to send another PR for this, or I can send it if not.

@patak-dev patak-dev added feat: create-vite create-vite package p2-nice-to-have Not breaking anything but nice to have (priority) labels Jul 13, 2024
@rschristian
Copy link
Contributor Author

Yep, can do! I held off as we don't yet support passing in the target directory, which means users will need to type the name in twice. Obviously a very quick fix, I just haven't had a moment to do it yet.

Was going to make a separate PR here once that was added & released.

@patak-dev patak-dev merged commit 564c8f4 into vitejs:main Jul 14, 2024
11 checks passed
@rschristian rschristian deleted the chore/create-preact-recommendation branch July 14, 2024 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: create-vite create-vite package p2-nice-to-have Not breaking anything but nice to have (priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants