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

Accept float values for socket_timeout #1299

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

kajinamit
Copy link
Contributor

@kajinamit kajinamit commented Mar 20, 2023

The socket_timeout argument of redis client supports not only integers
but also floats. This allows users to pass float values[1].

[1] https://github.com/redis/redis-py/blob/1b2f408259/redis/connection.py#L1206

@kajinamit kajinamit force-pushed the socket_timeout-float branch from 9af51c3 to 414f01b Compare March 20, 2023 07:44
@kajinamit kajinamit changed the title Socket timeout float Accept float values for socket_timeout Mar 20, 2023
@tobias-urdin
Copy link
Contributor

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

rebase

❌ Unable to rebase: user tobias-urdin is unknown.

Please make sure tobias-urdin has logged in Mergify dashboard.

@tobias-urdin
Copy link
Contributor

@Mergifyio rebase

The socket_timeout argument of redis client supports not only integers
but also floats. This allows users to pass float values[1].

[1] https://github.com/redis/redis-py/blob/1b2f408259/redis/connection.py#L1206
@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2023

rebase

✅ Branch has been successfully rebased

@tobias-urdin tobias-urdin force-pushed the socket_timeout-float branch from 414f01b to 456f57b Compare August 1, 2023 07:49
@tobias-urdin tobias-urdin merged commit 7b1bc65 into gnocchixyz:master Aug 1, 2023
@kajinamit kajinamit deleted the socket_timeout-float branch August 1, 2023 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants