Skip to content

ci: cache pnpm dependencies and reduce redundant workflows - #156

Merged
luojiyin1987 merged 3 commits into
masterfrom
ci/cache-concurrency-dedupe
Nov 4, 2026
Merged

luojiyin1987 merged 3 commits into
masterfrom
ci/cache-concurrency-dedupe

Conversation

@luojiyin1987

@luojiyin1987 luojiyin1987 commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Feature branch pushes trigger both push and pull request workflows. Lint installs and builds across three Node.js versions.
The workflow also lacks pnpm store caching and cancellation for older runs.

Changes

  • Cache the pnpm store through actions/setup-node in Test and Lint.
  • Add a concurrency group that cancels older runs for the same pull request or ref.
  • Run push workflows only for master.
  • Run Lint once on Node.js 22 without a build step.
  • Set the supported Node.js range to 22 or newer in the package and README.
  • Run Test on Node.js 22 and 24 with all existing checks.

Validation

  • Parsed the workflow YAML and checked its trigger, matrix, step order, cache path, and Lint steps.
  • pnpm install --frozen-lockfile passed.
  • pnpm run lint passed.
  • pnpm run build passed. API reports did not change.
  • pnpm run test:built passed: 2026 built tests and 11 source tests.
  • git diff --check passed.
  • The current build run passed Lint and Node.js 22/24.
  • Node.js 22 completed coverage and both benchmark smoke checks. Its pnpm cache restored successfully.
  • The current source-map validation run passed downstream compatibility checks.
  • The branch has one workflow per pull request commit and no push workflow.

Run cancellation needs a later overlapping workflow run.
pnpm 11 requires Node.js 22.13 or newer. The project now excludes Node.js 20.
No sensitive data was added or redacted.

@luojiyin1987
luojiyin1987 merged commit bc59c11 into master Nov 4, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant