-
Notifications
You must be signed in to change notification settings - Fork 498
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
Rename: forking deps/hiredis #192
Comments
18 tasks
i can't assign this issue to myself, please help me @zuiderkwast |
There is no decision about this yet, so please hold off. Maybe we'll do it in a future release I would say. There are some drawbacks, like being unable to update it if if hiredis is improved upstream. It's still BSD. |
i see, let me know and then we can update, i will be happy to help. |
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is no decision about this yet.
It is used by valkey-cli and valkey-benchmark.
As long as hiredis is vendored, we shall probably avoid changing anything in it, since it would make our vendored version diverge from the upstream hiredis. We can consider forking it though.
The text was updated successfully, but these errors were encountered: