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

Use npx and pnpx in CLI next steps instead of npm and pnpm #4273

Closed
wants to merge 1 commit into from
Closed

Use npx and pnpx in CLI next steps instead of npm and pnpm #4273

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 10, 2024

Closes: #4272

What are the changes and their implications?

This PR changes the blitz new CLI to mention the correct run commands once the app is built:

npx blitz dev
pnpx blitz dev

// instead of

npm blitz dev
pnpm blitz dev

It does this by using replace().

Bug Checklist

  • Changeset added (run pnpm changeset in the root directory)
  • Integration test added (see test docs if needed)

Feature Checklist

- use npx/pnpx instead of npm/pnpm
@ghost ghost requested a review from flybayer as a code owner January 10, 2024 00:27
Copy link

changeset-bot bot commented Jan 10, 2024

⚠️ No Changeset found

Latest commit: 6cc1758

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@siddhsuresh
Copy link
Member

Closing based on #4272 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

When Using npx, Blitz Incorrectly References npm
2 participants