fix(deploy): include all deps by default (#5040)

close #5035
pnpm · Jul 17, 2022 · 31e73ba · 31e73ba
1 parent 0e9fa30
commit 31e73ba
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 2 deletions.
diff --git a/.changeset/cool-chefs-march.md b/.changeset/cool-chefs-march.md
@@ -0,0 +1,6 @@
+---
+"@pnpm/plugin-commands-deploy": patch
+"pnpm": patch
+---
+
+`pnpm deploy` should include all dependencies by default [#5035](https://github.com/pnpm/pnpm/issues/5035).
diff --git a/packages/plugin-commands-deploy/src/deploy.ts b/packages/plugin-commands-deploy/src/deploy.ts
@@ -8,6 +8,8 @@ import PnpmError from '@pnpm/error'
 import rimraf from '@zkochan/rimraf'
 import renderHelp from 'render-help'
 
+export const shorthands = install.shorthands
+
 export function rcOptionsTypes () {
   return install.rcOptionsTypes()
 }
@@ -21,9 +23,29 @@ export const commandNames = ['deploy']
 export function help () {
   return renderHelp({
     description: 'Experimental! Deploy a package from a workspace',
-    descriptionLists: [],
     url: docsUrl('deploy'),
     usages: ['pnpm --filter=<deployed project name> deploy <target directory>'],
+    descriptionLists: [
+      {
+        title: 'Options',
+        list: [
+          {
+            description: "Packages in `devDependencies` won't be installed",
+            name: '--prod',
+            shortAlias: '-P',
+          },
+          {
+            description: 'Only `devDependencies` are installed regardless of the `NODE_ENV`',
+            name: '--dev',
+            shortAlias: '-D',
+          },
+          {
+            description: '`optionalDependencies` are not installed',
+            name: '--no-optional',
+          },
+        ],
+      },
+    ],
   })
 }
 
@@ -61,7 +83,6 @@ export async function handler (
     },
     frozenLockfile: false,
     preferFrozenLockfile: false,
-    dev: false,
     virtualStoreDir: path.join(deployDir, 'node_modules/.pnpm'),
     modulesDir: path.relative(deployedDir, path.join(deployDir, 'node_modules')),
   })

diff --git a/packages/plugin-commands-deploy/test/deploy.test.ts b/packages/plugin-commands-deploy/test/deploy.test.ts
@@ -48,6 +48,8 @@ test('deploy', async () => {
     ...DEFAULT_OPTS,
     allProjects,
     dir: process.cwd(),
+    dev: false,
+    production: true,
     recursive: true,
     selectedProjectsGraph,
     workspaceDir: process.cwd(),