adds protocol support for `redis` and `rediss`

[kjaymiller]

Pull Request check-list

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

• [X] Do tests and lints pass with this change?
• [X] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
• [X] 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)?
• [X] 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 :white_check_mark:

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

Additional details and impacted files

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[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