From 1dce6d3bd291c41281ceac4927470b78770aaa4e Mon Sep 17 00:00:00 2001 From: donjerick Date: Thu, 4 Sep 2025 23:37:24 +0800 Subject: [PATCH 1/2] chore: optimize CI and gitignore for pnpm workflow - Remove npm cache from CI workflow - Clean up gitignore by removing unnecessary lock file patterns - Keep pnpm-lock.yaml as the primary lock file - Streamline CI for better performance --- .github/workflows/ci.yml | 1 - .gitignore | 5 ----- 2 files changed, 6 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d9244db..2152d25 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -22,7 +22,6 @@ jobs: uses: actions/setup-node@v4 with: node-version: ${{ matrix.node-version }} - cache: 'npm' # Use npm in CI for maximum reliability - name: Install dependencies diff --git a/.gitignore b/.gitignore index 06abe34..96e1bad 100644 --- a/.gitignore +++ b/.gitignore @@ -21,11 +21,6 @@ Thumbs.db .DS_Store Thumbs.db -# Package manager files (keep one, ignore others) -package-lock.json -yarn.lock -bun.lockb - # Build outputs *.tsbuildinfo .nyc_output/ \ No newline at end of file From 0245b11c75cbac8d2f8283fcb42b3bfd961a4e20 Mon Sep 17 00:00:00 2001 From: donjerick Date: Thu, 4 Sep 2025 23:46:55 +0800 Subject: [PATCH 2/2] fix: use npm install instead of npm ci in GitHub Actions - Change from npm ci to npm install to work with pnpm-lock.yaml - Keep single lock file approach (pnpm-lock.yaml only) - Avoid developer confusion from multiple lock files - CI will generate node_modules from existing lock file - Apply fix to both test and publish jobs --- .github/workflows/ci.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 2152d25..b45a64d 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -23,9 +23,9 @@ jobs: with: node-version: ${{ matrix.node-version }} - # Use npm in CI for maximum reliability + # Use npm install in CI - name: Install dependencies - run: npm ci + run: npm install - name: Run linting run: npm run lint @@ -51,7 +51,7 @@ jobs: registry-url: 'https://registry.npmjs.org' - name: Install dependencies - run: npm ci + run: npm install # Extract version from GitHub release tag - name: Update package.json version