Bar Shaul

Results 10 comments of Bar Shaul

Added type hints + moved CredentialsProvider to a separate file

@chayim @dvora-h - Finished round, ready for review :). Tests failed here from some internal error, in my fork it seems that tests are passing: https://github.com/barshaul/redis-py/pull/16#partial-pull-merging

@chayim Finished round. I left some of the reviews open with questions

> @barshaul See tests failure @dvora-h fixed

I've never worked with poetry before, but it sounds interesting. Will this migration have any downsides?

> Sorry. The channel name you proposed was fine `__cluster__:moved` but I was questioning if we needed to have the remaining string be consistent with the moved response. `-MOVED` is...

@hpatro @VoletiRam Does it filter out failed and loading nodes? It would be ideal if the client's topology map could be solely based on the results of this command, eliminating...

@zhangskz Are there any updates on this?