Skip to content

chore: switch from npm to pnpm #360

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nicknisi merged 3 commits into from
Jan 14, 2026
Merged

chore: switch from npm to pnpm #360

nicknisi merged 3 commits into from
Jan 14, 2026

Conversation

@nicknisi
Copy link
Member

@nicknisi nicknisi commented Jan 10, 2026

Summary

  • Migrates package manager from npm to pnpm
  • Updates CI and release workflows to use pnpm
  • Adds @jest/globals and tslib as explicit devDependencies (required due to pnpm's stricter dependency resolution)
  • Updates internal scripts to use pnpm run instead of npm run

Test plan

  • pnpm install succeeds
  • pnpm build succeeds
  • pnpm test passes (294/294 tests)

@nicknisi nicknisi requested a review from a team as a code owner January 10, 2026 16:31
@nicknisi nicknisi requested a review from cmatheson January 10, 2026 16:31
gjtorikian
gjtorikian approved these changes Jan 14, 2026
View reviewed changes
Copy link

@gjtorikian gjtorikian left a comment

Choose a reason for hiding this comment

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

pretty straightforward, lgtm!

@nicknisi nicknisi merged commit f5204e8 into main Jan 14, 2026
8 checks passed
@nicknisi nicknisi deleted the nicknisi/pnpm branch January 14, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gjtorikian gjtorikian gjtorikian approved these changes

@cmatheson cmatheson Awaiting requested review from cmatheson cmatheson is a code owner automatically assigned from workos/typescript

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nicknisi @gjtorikian