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

[create-astro] verify connectivity and --template #8102

Merged
merged 6 commits into from
Aug 16, 2023
Merged

Conversation

natemoo-re
Copy link
Member

Changes

  • Improved DX for common create-astro issues

If user is not connected to the internet, bail out early

internet connectivity error

If --template does not exist, bail out early

missing template error

For some history on the --template issue, giget doesn't check if a template exists before downloading. We have an existing check after cloning for empty contents, but this PR adds an up-front check that the given repo/subpath is not 404

Testing

Added verify.test.js suite

Docs

N/A, this is a DX improvement

@changeset-bot
Copy link

changeset-bot bot commented Aug 16, 2023

🦋 Changeset detected

Latest commit: 514a203

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: create-astro Related to the `create-astro` package (scope) label Aug 16, 2023
@natemoo-re natemoo-re merged commit e6e1de4 into main Aug 16, 2023
@natemoo-re natemoo-re deleted the feat/cli-verify branch August 16, 2023 19:37
This was referenced Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: create-astro Related to the `create-astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants