-
Notifications
You must be signed in to change notification settings - Fork 533
Setting a timeout on git subprocesses with the v2 pre-release version #690
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
Comments
The timeout behavior has not been exposed to the interface yet. Let's leave this issue open as a feature request and I'll address it as soon as time allows. |
@gregfletch and @nckbowen (and anyone else who is interested) I have proposed how the timeout feature would work in #692. Would appreciate your feedback. It is just the description of the implementation in an updated project README.md. Implementation will follow depending on feedback received. |
LGTM, thanks for adding this so quickly @jcouball! |
@jcouball Thanks for taking up this feature. We see that the PR for adding the timeout has been merged. Would it be possible for you to release an updated new version for v2 pre-release so we can give this feature a try? |
@gregfletch and @rwabhinav I have just pushed git 2.0.0.pre2 to rubygems.org. I am interested to hear your feedback. Thanks for helping out! 🌟 ⭐ |
Subject of the issue
I am currently testing the v2 pre-release version of the gem and am particularly interested in being able to set a timeout on git subprocesses which from the sounds of #684 should be possible with the pre-release version of the gem. However, it is not clear from the documentation how one sets/passes a timeout value. For example, if I wanted to call
Git.clone
, it is not clear from the method documentation how or if it is possible to provide a timeout value. Can you clarify if this functionality is supported with v2.0.0.pre1, and if so how (looking at the method documentation forclone
, the options hash seems to be for the options provided to the git command)?Your environment
Steps to reproduce
N/A, doc improvement/clarification.
Expected behaviour
Clarification as to whether or not support for setting timeouts is available currently with the pre-release gem version and if it is, clarification as to how timeouts are set.
Actual behaviour
What actually happened?
The text was updated successfully, but these errors were encountered: