Skip to content

Fix bin path on Windows, fixes #2565 - #3216

Closed
saschanaz wants to merge 1 commit into
yarnpkg:masterfrom
saschanaz:patch-1
Closed

saschanaz wants to merge 1 commit into
yarnpkg:masterfrom
saschanaz:patch-1

Conversation

@saschanaz

Copy link
Copy Markdown

Summary

yarn global add (package name) installs a package on %LOCALAPPDATA%\Yarn\config\global\node_modules\.bin but the installer adds wrong path. This PR fixes it.

Test plan

No test, can you help me if a test for this is required?

@saschanaz

Copy link
Copy Markdown
Author

It seems that master already fails..

@Daniel15

Copy link
Copy Markdown
Member

I think #3233 will actually fix this.

@saschanaz

Copy link
Copy Markdown
Author

@Daniel15 Great :D

@saschanaz saschanaz closed this Apr 24, 2017
@saschanaz
saschanaz deleted the patch-1 branch April 24, 2017 10:07
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