From a6f90078810771c2f124e982c0f258d529217c64 Mon Sep 17 00:00:00 2001 From: Mikita Kaliasneu Date: Sun, 25 May 2025 14:52:13 +0300 Subject: [PATCH] docs: updated comments to reflect pnpm usage instead of yarn --- packages/integration-tests/tools/pack-packages.ts | 8 ++++---- tools/scripts/postinstall.mts | 3 +-- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/packages/integration-tests/tools/pack-packages.ts b/packages/integration-tests/tools/pack-packages.ts index fceb268f856..5c97688b1ff 100644 --- a/packages/integration-tests/tools/pack-packages.ts +++ b/packages/integration-tests/tools/pack-packages.ts @@ -130,13 +130,13 @@ export const setup = async (project: TestProject): Promise => { ), { encoding: 'utf-8' }, ); - // TODO!: Rewrite - // We install the tarballs here once so that yarn can cache them globally. + + // We install the tarballs here once so that pnpm can cache them globally. // This solves 2 problems: // 1. Tests can be run concurrently because they won't be trying to install // the same tarballs at the same time. - // 2. Installing the tarballs for each test becomes much faster as Yarn can - // grab them from the global cache folder. + // 2. Installing the tarballs for each test becomes much faster as pnpm can + // reuse them from its global content-addressable store. await execFile('pnpm', ['install', '--no-frozen-lockfile'], { cwd: temp, shell: true, diff --git a/tools/scripts/postinstall.mts b/tools/scripts/postinstall.mts index 8d320e2e350..2a175b077f3 100644 --- a/tools/scripts/postinstall.mts +++ b/tools/scripts/postinstall.mts @@ -2,11 +2,10 @@ import { $ as $_config } from 'execa'; const $ = $_config({ env: { - // TODO!: Check it out and, if necessary, rewrite it /** * Do not apply the special GitHub Actions group markers within the * postinstall logging, it cannot work correctly when nested within - * yarn's output and therefore just adds visual noise. + * pnpm's output and therefore just adds visual noise. */ NX_SKIP_LOG_GROUPING: 'true', },