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(turborepo): Use which crate to resolve yarn binary #5001
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
9 Ignored Deployments
|
4f0d37a
to
67e58e6
Compare
🟢 CI successful 🟢Thanks |
@@ -76,7 +77,8 @@ fn execute_git_command( | |||
args: &[&str], | |||
pathspec: &str, | |||
) -> Result<Vec<u8>, Error> { | |||
let mut command = Command::new("git"); | |||
let git_binary = which("git")?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're going to use git
a lot. We should perhaps memoize this and make it easy to use. Can do in a follow-up though.
@@ -36,3 +36,48 @@ Set package manager to pnpm in package.json | |||
Run test run | |||
$ ${TURBO} run build --__test-run | jq .package_manager | |||
"pnpm" | |||
|
|||
Clear package manager field in package.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to verify this manually on windows, these tests don't run there
97134bd
to
2631b61
Compare
Cargo.toml
Outdated
@@ -197,3 +197,4 @@ tracing-subscriber = "0.3.16" | |||
url = "2.2.2" | |||
urlencoding = "2.1.2" | |||
webbrowser = "0.8.7" | |||
which = "4.4.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the lack of a newline here is hitting the linter
Description
Fixes #4940 by using
which
crate to resolve yarn binary. That way we use the correct extension, i.e.yarn.cmd
on Windows.Testing Instructions
Tested manually on Windows. Will add more integration tests