redis.Redis is no longer documented, only redis.client.redis

[Zac-HD]

We noticed this because the docs build for Hypothesis now warns that our intersphinx link to redis.Redis no longer resolves.

Is this change intentional and importing from redis.client now best-practice? If so, a mention in the release notes would be helpful! If not, it would be great to document the shorter alias again 😁

[chayim]

@Zac-HD just to make sure is this the project? redis.Redis is definitely the right thing to import, though the files have been in redis.client for ... as long as I've been involved.

It looks like by expanding documentation for 4.1.0 we inadvertently broke yours. Can you pass me a link - or something we can look at?

[Zac-HD]

That's right; this docstring ends up on this page of our docs (and here's a failing CI build).

Sounds like this is just a Sphinx config thing then; changing .. autoclass:: redis.client.Redis to .. autoclass:: redis.Redis might even be sufficient?

[chayim]

@Zac-HD Perfect, thank you for the pointers. Unfortunately this would be going into master so it would appear in latest rather that stable. I'll cross-link the PR that closes this.