typing for client __init__

[Vulwsztyn]

Pull Request check-list

Please make sure to review and check all of these items:

• Do tests and lints pass with this change?
• Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
• Is the new or changed code fully tested?
• Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
• Is there an example added to the examples folder (if applicable)?
• Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

Description of change

I only added typings to __init__ method of client:Redis class

I realise this exists: #3252

But there were some comments and it is dead since the end of May, while I hope this PR introduces so little change that it is instantly mergeable.

[vladvildanov]

@Vulwsztyn Tests are failing, cause of conditional OpenSSL dependency

[Vulwsztyn]

@vladvildanov thanks I'll make it unconditional

[vladvildanov]

@Vulwsztyn Thank you for your contribution!

[Vulwsztyn]

@vladvildanov I changed the typing to string literals so that it doesn't require imports and removed the conditional imports

[Vulwsztyn]

@vladvildanov Any insight as to why the tests are failing now? Should I do something?

[Vulwsztyn]

@vladvildanov I fixed the linting issues. Sorry it took so long.

> invoke linters
All done! ✨ 🍰 ✨
134 files would be left unchanged.
Running flynt v.0.69
Running flynt in dry-run mode. No files will be changed.

Flynt run has finished. Stats:

Execution time:                            2.488s
Files modified:                            0

_-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_.
Please run your tests before committing. Did flynt get a perfect conversion? give it a star at: 
~ https://github.com/ikamensh/flynt ~
Thank you for using flynt. Upgrade more projects and recommend it to your colleagues!
> invoke linters
All done! ✨ 🍰 ✨
134 files would be left unchanged.
Running flynt v.0.69
Running flynt in dry-run mode. No files will be changed.

Flynt run has finished. Stats:

Execution time:                            2.488s
Files modified:                            0

_-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_._-_.
Please run your tests before committing. Did flynt get a perfect conversion? give it a star at: 
~ https://github.com/ikamensh/flynt ~
Thank you for using flynt. Upgrade more projects and recommend it to your colleagues!

[Vulwsztyn]

rebased unto master

[vladvildanov]

@Vulwsztyn OpenSSL is still an issue if you have a look in CI logs

[Vulwsztyn]

@vladvildanov Thanks for the ping.

I don't get something here. In this line

redis-py/redis/connection.py

Line 870 in 7d73d74

import OpenSSL

in this project we import OpenSSL, but it is not a dependency of this project? Should it remain so?

[vladvildanov]

@Vulwsztyn https://redis-py.readthedocs.io/en/stable/examples/ssl_connection_examples.html#Connect-via-SSL,-validate-OCSP-stapled-certificates

[Vulwsztyn]

Removed the typing requiring openssl

[Vulwsztyn]

Could you try running the tests please?

[Vulwsztyn]

I refixed lint

[Vulwsztyn]

I see that all checks are passing. Would it be possible to merge this?

[vladvildanov]

@Vulwsztyn waiting for checks after latest sync with master and good to go