-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Wrong starter template description (and typo) #2372
Comments
Hmm. Interesting I’m not actually clear what the difference between those two repos is that makes one fail, I think theoretically they should both work. It might be the branch name? Noticing that the one that fails uses |
@delucis you're right. I added a Is this worth documenting? Or perhaps |
Let me check with the core team. Maybe we can use the default branch in |
As a side note, it looks like you can also specify the branch name after a npm create astro@latest -- --template ohansemmanuel/astrojs-ditch-the-runtime-react#master It looks like this could be a limitation of the underlying tool Astro uses: https://github.com/unjs/giget/ When parsing the template location, they look for |
That makes sense. It does work with the branch name after a Thanks for the input @delucis - Please feel free to either close this issue or consider a possible next step: documenting this limitation (with a small note after the instructions) |
Opened #2413 to address this. Thanks for raising this issue @ohansemmanuel! |
What?
The following is a rather ambiguous (slightly misleading) sentence as it gives the impression you can take any public Github astro project and start a new project based on it (i.e., copy the files as a "template")
In reality, this does not seem to be the case.
How?
it appears (as I suspected), the creation from an "existing Github repository" command/description only works with existing starter templates
e.g.,
✅ this works:
❌ this does not work:
Suggestion
If the assumption above is correct i.e., this works for existing templates, then it's perhaps worth reconsidering rewording the sentence to:
The text was updated successfully, but these errors were encountered: