-
Notifications
You must be signed in to change notification settings - Fork 112
Modify retry() to take a callable and enforce the timeout #372
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
Conversation
|
@categulario |
48b32b3 to
bb8ddf0
Compare
Codecov Report
@@ Coverage Diff @@
## master #372 +/- ##
==========================================
+ Coverage 88.13% 88.54% +0.41%
==========================================
Files 15 16 +1
Lines 1500 1563 +63
==========================================
+ Hits 1322 1384 +62
- Misses 178 179 +1
Continue to review full report at Codecov.
|
|
In the latest commits I even added three tests |
b0489b9 to
59f2723
Compare
|
@categulario merged in I will need to drop python 3.7 and 3.8 in tmuxp since that's happening in libtmux before 0.12.0a0 can be added to tmuxp: tmux-python/tmuxp#763 |
|
tmux-python/tmuxp#776 has a concept of |
|
@categulario As a next step you are welcome to make a PR moving if you do have something started let me know, if not i will get to it during the weekend. I have an example at tmux-python/tmuxp#776 |
Is that necessary? python 3.7 is still maintained. and 3.8 comes in ubuntu 20.04. What exactly makes libtmux incompatible?
Sure thing! |
I thought about that. I can look at reversing it, but I'd need to do it in libtmux first. See #374 |
|
I can expand #374 to cover 3.7 (this would need to happen during the weekend though) |
|
@categulario libtmux and tmuxp support python 3.7 and 3.8 again ✅ Do you still want to move the tests over to P.S. It's open source, none of us our paid, but the reason I hurry is the warnings due to the |
P.S. And the remainder - in python 3.10 - are from the user
That sounds good to me |

Previous implementation is equivalent to
return True. This implementation, while API incompatible, is easy to use (I would modify the corresponding code in tmuxp myself) and enforces the timeout via an exception, which would help debugging since it provide the specific cause.API docs updated accordingly. Environment variable preserved.
fixes #368 and addresses tmux-python/tmuxp#704 (comment) , it would also help with tmux-python/tmuxp#620