Skip to content

chore: pin pnpm via packageManager#123

Merged
mscolnick merged 1 commit into
mainfrom
chore/pnpm-version-pin
May 12, 2026
Merged

chore: pin pnpm via packageManager#123
mscolnick merged 1 commit into
mainfrom
chore/pnpm-version-pin

Conversation

@mscolnick
Copy link
Copy Markdown
Contributor

Adds "packageManager": "pnpm@10.28.2" to package.json and drops the redundant version: arg from pnpm/action-setup in CI.

The action now reads the version from packageManager, giving a single source of truth shared between local dev and CI. Matches the convention used in marimo and marimo-lsp.

Adds packageManager: pnpm@10.28.2 to package.json and drops the
redundant `version:` arg from pnpm/action-setup. The action now reads
the version from packageManager, giving a single source of truth shared
between local dev and CI.
Copilot AI review requested due to automatic review settings May 12, 2026 15:00
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Pins the project’s pnpm version via package.json’s packageManager field and simplifies GitHub Actions workflows to rely on that single source of truth, aligning local development and CI.

Changes:

  • Add "packageManager": "pnpm@10.28.2" to package.json.
  • Remove the redundant with: version: 10 configuration from pnpm/action-setup in CI/release/demo workflows.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Declares the pinned pnpm version via packageManager.
.github/workflows/test.yml Drops explicit pnpm version so CI reads from packageManager.
.github/workflows/release.yml Drops explicit pnpm version so release workflow reads from packageManager.
.github/workflows/demo.yml Drops explicit pnpm version so demo workflow reads from packageManager.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mscolnick mscolnick merged commit 3388dee into main May 12, 2026
8 checks passed
@mscolnick mscolnick deleted the chore/pnpm-version-pin branch May 12, 2026 15:13
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.

2 participants