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

fix(cli): Suggest pnpm dlx instead of pnpx #11183

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

66Leo66
Copy link
Contributor

@66Leo66 66Leo66 commented Jun 4, 2024

Changes

  • pnpx was removed in pnpm v7 and replaced by pnpm exec and pnpm dlx. (feat: dlx pnpm/pnpm#3652)
  • pnpm dlx works the same as yarn dlx.
  • Astro update check's command sample should suggest pnpm dlx

Before:

 update  ▶ New version of Astro available: 4.9.2
  Run pnpx @astrojs/upgrade to update

After:

 update  ▶ New version of Astro available: 4.9.2
  Run pnpm dlx @astrojs/upgrade to update

Testing

N/A

Docs

N/A

pnpx was removed in pnpm v7 and replaced by pnpm exec and pnpm dlx.

pnpm dlx works the same as yarn dlx.
Copy link

changeset-bot bot commented Jun 4, 2024

🦋 Changeset detected

Latest commit: 9569a86

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: astro Related to the core `astro` package (scope) label Jun 4, 2024
@matthewp matthewp merged commit 3cfa2ac into withastro:main Jun 4, 2024
13 checks passed
@astrobot-houston astrobot-houston mentioned this pull request Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: astro Related to the core `astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants