Skip to content

refactor: move from npm to pnpm#8015

Draft
joshblack wants to merge 16 commits into
mainfrom
refactor/update-to-pnpm
Draft

refactor: move from npm to pnpm#8015
joshblack wants to merge 16 commits into
mainfrom
refactor/update-to-pnpm

Conversation

@joshblack

@joshblack joshblack commented Jun 17, 2026

Copy link
Copy Markdown
Member

Update the project to use pnpm instead of npm. This allows for improved dependency resolution in the monorepo (namely around hoisting) and install performance locally and in CI.

Changelog

New

Changed

  • Update workflows to use pnpm
  • Update package.json and scripts to use pnpm
  • Add missing dependencies in various package.json files to support correct node_modules layout

Removed

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None; if selected, include a brief description as to why

This is a change to the package manager of the project only

@changeset-bot

changeset-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 35495ef

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

@github-actions github-actions Bot added the staff Author is a staff member label Jun 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Action required

👋 Hi, this pull request contains changes to the source code that github/github-ui depends on. If you are GitHub staff, test these changes with github/github-ui using the integration workflow. Check the integration testing docs for step-by-step instructions. Or, apply the integration-tests: skipped manually label to skip these checks.

To publish a canary release for integration testing, apply the Canary Release label to this PR.

@github-actions github-actions Bot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Jun 17, 2026
@github-actions github-actions Bot requested a deployment to storybook-preview-8015 June 17, 2026 19:57 Abandoned
@joshblack joshblack changed the title Refactor/update to pnpm refactor: move from npm to pnpm Jun 17, 2026
@joshblack joshblack added the skip changeset This change does not need a changelog label Jun 18, 2026
@github-actions github-actions Bot requested a deployment to storybook-preview-8015 June 18, 2026 16:23 Abandoned
@github-actions github-actions Bot temporarily deployed to storybook-preview-8015 June 18, 2026 16:57 Inactive
@joshblack

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits June 23, 2026 20:32
Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>
Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Conflicts are resolved and merged with latest main in 76bddca (with a small follow-up workflow text fix in 379abc5).

@github-actions github-actions Bot requested a deployment to storybook-preview-8015 June 23, 2026 20:38 Abandoned
@joshblack

Copy link
Copy Markdown
Member Author

@copilot fix CI

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@copilot fix CI

Fixed in 35495ef by regenerating pnpm-lock.yaml to match merged package manifests. This resolves the pnpm install --frozen-lockfile CI failures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm skip changeset This change does not need a changelog staff Author is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants