Skip to content

feat: Add safe Yarn PnP support#270

Draft
bmulholland wants to merge 3 commits into
unjs:mainfrom
bmulholland:main
Draft

feat: Add safe Yarn PnP support#270
bmulholland wants to merge 3 commits into
unjs:mainfrom
bmulholland:main

Conversation

@bmulholland

@bmulholland bmulholland commented Jul 26, 2024

Copy link
Copy Markdown

Hopefully resolved #221 :)

Opening this PR as a draft. I've tested it on my code (full disclaimer: only as a yarn patch, not building this exact code from scratch), and it works well. Looking for feedback on approach, code organization, etc.

@CarsonF

CarsonF commented Feb 13, 2025

Copy link
Copy Markdown

Thank you for the patch @bmulholland this worked for me too.
Hopefully this can get merged/released.

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.

doesn't work with yarn pnp and esm-only modules with import condition

2 participants