Skip to content

Commit

Permalink
fix(deploy): include all deps by default
Browse files Browse the repository at this point in the history
close #5035
  • Loading branch information
zkochan committed Jul 16, 2022
1 parent 0e9fa30 commit f0c6a79
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
6 changes: 6 additions & 0 deletions .changeset/cool-chefs-march.md
Original file line number Diff line number Diff line change
@@ -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).
1 change: 0 additions & 1 deletion packages/plugin-commands-deploy/src/deploy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,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')),
})
Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-commands-deploy/test/deploy.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,8 @@ test('deploy', async () => {
...DEFAULT_OPTS,
allProjects,
dir: process.cwd(),
dev: false,
production: true,
recursive: true,
selectedProjectsGraph,
workspaceDir: process.cwd(),
Expand Down

0 comments on commit f0c6a79

Please sign in to comment.