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

Setting a timeout on git subprocesses with the v2 pre-release version #690

Closed
gregfletch opened this issue Feb 1, 2024 · 5 comments
Closed

Comments

@gregfletch
Copy link

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 for clone, the options hash seems to be for the options provided to the git command)?

Your environment

  • version of git: 2.43.0 and ruby-git: 2.0.0.pre1
  • version of ruby: 3.3.0

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?

@jcouball
Copy link
Member

jcouball commented Feb 1, 2024

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.

@jcouball
Copy link
Member

jcouball commented Feb 2, 2024

@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.

@gregfletch
Copy link
Author

LGTM, thanks for adding this so quickly @jcouball!

@rwabhinav
Copy link

@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?

@jcouball
Copy link
Member

@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! 🌟 ⭐

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

No branches or pull requests

3 participants