Skip to content

ci: fix pnpm/setup config (invalid require-lockfile input) - #54

Merged
edoardocavazza merged 6 commits into
mainfrom
claude/pnpm-setup-action-upgrade-5kdxj6
Sep 4, 2026
Merged

edoardocavazza merged 6 commits into
mainfrom
claude/pnpm-setup-action-upgrade-5kdxj6

Conversation

@edoardocavazza

Copy link
Copy Markdown
Member

Summary

The previous pnpm/setup migration (#53, already merged) used a require-lockfile input that does not actually exist on pnpm/setup@v2 (valid inputs are version, dest, runtime, cache, cache-dependency-path, package-json-file, install, token), which was silently ignored with a CI warning and left installs unpinned to the lockfile. This PR:

  • Replaces require-lockfile: true with install: false on the pnpm/setup step plus an explicit pnpm install --frozen-lockfile step, restoring the original frozen-lockfile behavior.
  • Re-adds runtime: node@latest on every pnpm/setup step.

Test plan

  • Confirm CI (lint/build/test jobs, release, pages) passes on this branch

🤖 Generated with Claude Code

https://claude.ai/code/session_01NM1AyJH42derNim3BmJpVB


Generated by Claude Code

@changeset-bot

changeset-bot Bot commented Sep 4, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 830c614

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@edoardocavazza
edoardocavazza merged commit 5223425 into main Sep 4, 2026
3 checks passed
@edoardocavazza
edoardocavazza deleted the claude/pnpm-setup-action-upgrade-5kdxj6 branch September 4, 2026 09:52
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