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

Fix switch statement exhaustiveness checking #304

Merged
merged 1 commit into from Oct 31, 2019

Conversation

GulajavaMinistudio
Copy link
Owner

  • fix(run): change run command to check cwd/node_modules/.bin for commands. Fixes run in workspaces.

This fixes a bug that was introduced in yarnpkg#6850 where the bin path was being built only from
config.lockfileFolder. However in workspaces, bins may not be hoisted to the workspace root,
causing bins to not be found. This change adds config.cwd to the bin search path, so the yarn run command will look in a workspace package's node_modules, as well as the workspace root.

fixes yarnpkg#7126

  • modify chagelog

Summary

Test plan

* fix(run): change run command to check cwd/node_modules/.bin for commands. Fixes run in workspaces.

This fixes a bug that was introduced in #6850 where the bin path was being built only from
`config.lockfileFolder`. However in workspaces, bins may not be hoisted to the workspace root,
causing bins to not be found. This change adds `config.cwd` to the bin search path, so the `yarn
run` command will look in a workspace package's node_modules, as well as the workspace root.

fixes #7126

* modify chagelog
@GulajavaMinistudio GulajavaMinistudio merged commit 4f17390 into GulajavaMinistudio:master Oct 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants