Skip to content

fix: use pnpm in release workflow instead of npm#167

Merged
AdamJ merged 1 commit into
mainfrom
claude/eloquent-noether-Olpo6
Jun 6, 2026
Merged

fix: use pnpm in release workflow instead of npm#167
AdamJ merged 1 commit into
mainfrom
claude/eloquent-noether-Olpo6

Conversation

@AdamJ

@AdamJ AdamJ commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • The Release workflow was failing immediately because actions/setup-node had cache: npm, which requires a package-lock.json — but this project uses pnpm (pnpm-lock.yaml only)
  • Replaced cache: npm with pnpm/action-setup@v4 for proper pnpm support
  • Switched npm ci to pnpm install --frozen-lockfile
  • Fixed the version-bump commit step to stage pnpm-lock.yaml instead of package-lock.json

Root cause

##[error]Dependencies lock file is not found. Supported file patterns: package-lock.json,npm-shrinkwrap.json,yarn.lock

Failed run: https://github.com/AdamJ/TimeTrackerPro/actions/runs/27069250200

Test plan

  • Merge this PR — the push to main will trigger a new Release workflow run with the corrected pnpm setup

Generated by Claude Code

The release workflow was failing because setup-node had cache: npm but
the project uses pnpm (only pnpm-lock.yaml exists, no package-lock.json).
Switch to pnpm/action-setup and pnpm install --frozen-lockfile.
@github-actions github-actions Bot added actions For all items related to GitHub, deployment, and release automations and actions. automation For items related to release/documentation/workflow automations labels Jun 6, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying timetrackerpro with  Cloudflare Pages  Cloudflare Pages

Latest commit: e4e0a7c
Status: ✅  Deploy successful!
Preview URL: https://5f5131a3.timetrackerpro.pages.dev
Branch Preview URL: https://claude-eloquent-noether-olpo.timetrackerpro.pages.dev

View logs

@AdamJ AdamJ added dependencies Pull requests that update a dependency file and removed dependencies Pull requests that update a dependency file labels Jun 6, 2026
@github-project-automation github-project-automation Bot moved this to To triage in TimeTrackerPro Jun 6, 2026
@AdamJ AdamJ moved this from To triage to In review in TimeTrackerPro Jun 6, 2026
@AdamJ AdamJ merged commit 281a663 into main Jun 6, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in TimeTrackerPro Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

actions For all items related to GitHub, deployment, and release automations and actions. automation For items related to release/documentation/workflow automations

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants