Skip to content

Commit

Permalink
fix(core): pnpm exec check not matching for v7 (#10004)
Browse files Browse the repository at this point in the history
  • Loading branch information
muuvmuuv committed Apr 26, 2022
1 parent 869edb7 commit 68207ef
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions packages/nx/src/utils/package-manager.ts
Expand Up @@ -6,6 +6,7 @@ import { dirSync } from 'tmp';
import { promisify } from 'util';
import { readJsonFile, writeJsonFile } from './fileutils';
import { PackageJson } from './package-json';
import { gte } from 'semver';

const execAsync = promisify(exec);

Expand Down Expand Up @@ -61,9 +62,9 @@ export function getPackageManagerCommand(
list: 'yarn list',
}),
pnpm: () => {
const [major, minor] = getPackageManagerVersion('pnpm').split('.');
const pnpmVersion = getPackageManagerVersion('pnpm');
let useExec = false;
if (+major >= 6 && +minor >= 13) {
if (gte(pnpmVersion, '6.13.0')) {
useExec = true;
}
return {
Expand Down

1 comment on commit 68207ef

@vercel
Copy link

@vercel vercel bot commented on 68207ef Apr 26, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

nx-dev – ./

nx-dev-nrwl.vercel.app
nx-dev-git-master-nrwl.vercel.app
nx-five.vercel.app
nx.dev

Please sign in to comment.