Infrastructure/astro
0
Fork 0
mirror of https://github.com/withastro/astro.git synced 2025-03-31 23:31:30 -05:00
Code Issues Activity

Update install-package.ts (#11183)

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

pnpm dlx works the same as yarn dlx.
This commit is contained in:
Leo Li 2024-06-04 22:54:05 +08:00 • committed by GitHub
parent bf013cb3db
commit 3cfa2ac7e5
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/purple-pears-divide.md Normal file
View file

@ -0,0 +1,5 @@
---
'astro': patch
---
Suggest `pnpm dlx` instead of `pnpx` in update check.

4
packages/astro/src/cli/install-package.ts
View file

@ -99,7 +99,7 @@ function getInstallCommand(packages: string[], packageManager: string) {
}
/**
* Get the command to execute and download a package (e.g. `npx`, `yarn dlx`, `pnpx`, etc.)
* Get the command to execute and download a package (e.g. `npx`, `yarn dlx`, `pnpm dlx`, etc.)
* @param packageManager - Optional package manager to use. If not provided, Astro will attempt to detect the preferred package manager.
* @returns The command to execute and download a package
*/
@ -114,7 +114,7 @@ export async function getExecCommand(packageManager?: string): Promise<string> {
case 'yarn':
return 'yarn dlx';
case 'pnpm':
return 'pnpx';
return 'pnpm dlx';
case 'bun':
return 'bunx';
default: