adds protocol support for redis and rediss

[kjaymiller]

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

• Adds 'redis://' and 'rediss://' to the list of accepted protocols for both sync and async connection pools
• Updates tests to use parameterization so that tests for valkey:// and valkeys:// also test redis// and rediss://

resolves #26 #23

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@d116aa6). Learn more about missing BASE report.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage        ?   75.05%           
=======================================
  Files           ?      131           
  Lines           ?    34386           
  Branches        ?        0           
=======================================
  Hits            ?    25807           
  Misses          ?     8579           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[aiven-sal]

Thanks for the PR!
I have a couple of requests:

1. could you squash the lint changes in your second commit in the first one? It would make it easier for me to review commit-by-commit.
2. ignore the CHANGES file, it wasn't really used much recently and we have a nice bot that can summarize changes in releases from PRs. It isn't perfect, but for now let's just use that (you don't have to do anything).

[kjaymiller]

1. could you squash the lint changes in your second commit in the first one? It would make it easier for me to review commit-by-commit.

I hope I did this correctly

[kjaymiller]

@aiven-sal - Is there something on my end holding progress on this?

[aiven-sal]

@aiven-sal - Is there something on my end holding progress on this?

Nope, there is some related work going on in #29, so I'm waiting for it to get merged. Then this will need to be rebased (I'm sorry for the inconvenience).

[aiven-sal]

So #29 was merged. It should handle #23 and #25 .
This PR also contains the fix for #26.
To keep things tidier I'll close this PR, so you can open another one just for the #26 fix.
Thanks again and sorry for the delay. It was just easier to fix #23 and #25 together.