Skip to content
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.

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

Support PnP in 4 combinations of configurations #5

Merged
merged 1 commit into from May 27, 2019

Conversation

crubier
Copy link
Contributor

@crubier crubier commented May 26, 2019

Support Yarn PnP. Tested with 4 configurations combinations.

Fixes #4

@crubier crubier mentioned this pull request May 26, 2019
@crubier
Copy link
Contributor Author

crubier commented May 26, 2019

@rot1024 :-) Hope it helps

@rot1024
Copy link
Member

rot1024 commented May 27, 2019

Thank you for a PR! I tried your code and have confirmed it works correctly. It seems to be good.

I found yarnpkg/yarn#6643 to know about pnpapi, but I could not find documentation about it. How did you know about it?

@crubier
Copy link
Contributor Author

crubier commented May 27, 2019

@rot1024 rot1024 merged commit 14893d6 into reearth:master May 27, 2019
@rot1024
Copy link
Member

rot1024 commented May 27, 2019

Great. I have merged your PR and released v1.1.0!

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.

Crash when using yarn PnP
2 participants