Skip to content

docs: Add guide for testing SDK packages with pnpm link - #25512

Merged
andreiborza merged 1 commit into
developfrom
ab/docs-pnpm-link
Oct 19, 2026
Merged

andreiborza merged 1 commit into
developfrom
ab/docs-pnpm-link

Conversation

@andreiborza

Copy link
Copy Markdown
Member

What

Adds a guide for testing local SDK builds in a project with pnpm link, next to the existing yalc guide.

Why

pnpm link is the simplest way to test a local build, but we did not document how to link, rebuild, and unlink.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@andreiborza
andreiborza marked this pull request as ready for review October 19, 2026 09:28
@andreiborza
andreiborza requested review from JPeer264 and mydea October 19, 2026 09:29

@JPeer264 JPeer264 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for adding the guide

Comment thread docs/using-pnpm-link.md
pnpm unlink @sentry/node
```

In pnpm 10, `pnpm unlink` shows "Nothing to unlink" and does not remove the override. Remove the override

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

l: We're on v12. Is it still worth to mention pnpm v10?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I ran into this which is why it was there but fair. Removing it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

On second thought, just because we use 12 in the repo, doesn't mean external projects you are trying to link into use 12. Keeping it.

@andreiborza
andreiborza merged commit 36bd110 into develop Oct 19, 2026
39 checks passed
@andreiborza
andreiborza deleted the ab/docs-pnpm-link branch October 19, 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.

2 participants