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

Add option to set custom path to git binary #3393

Merged
merged 4 commits into from
Mar 19, 2021

Conversation

morhi
Copy link
Contributor

@morhi morhi commented Mar 18, 2021

Closes #3392

This is an idea to fix #3392 by making the path to the git binary configurable for a case where the git command is not directly available in the environment. Sorry for my bad english in the code comment 🙈

@jasonvarga
Copy link
Member

This seems fine to me. Are you planning to add anything else? You still have this PR marked as a draft. Feel free to switch it over if you're done.

@morhi
Copy link
Contributor Author

morhi commented Mar 18, 2021

Nope, that's all for this PR :) I am not sure if the code comment is good enough linguistically (because the config will be copied to the users repository), but if is, then I am happy with it, too ;)

@morhi morhi marked this pull request as ready for review March 18, 2021 14:16
@morhi
Copy link
Contributor Author

morhi commented Mar 18, 2021

Thanks, that comment makes more sense 🎉

@jesseleite
Copy link
Member

Works great, lovely! Thank you for PR.

@jesseleite jesseleite merged commit db16c34 into statamic:master Mar 19, 2021
@morhi morhi deleted the fix/git-binary branch March 20, 2021 19: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.

3 participants