Skip to content

Python: Replace instances of Redis with Valkey#2266

Merged
jonathanl-bq merged 7 commits intovalkey-io:mainfrom
Bit-Quill:python/lotjonat_valkey_274_rename_redis_to_valkey
Sep 11, 2024
Merged

Python: Replace instances of Redis with Valkey#2266
jonathanl-bq merged 7 commits intovalkey-io:mainfrom
Bit-Quill:python/lotjonat_valkey_274_rename_redis_to_valkey

Conversation

@jonathanl-bq
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Jonathan Louie <jonathanl@bitquilltech.com>
@jonathanl-bq jonathanl-bq added the python 🐍 Python wrapper label Sep 10, 2024
jonathanl-bq and others added 2 commits September 10, 2024 15:08
Signed-off-by: Jonathan Louie <jonathanl@bitquilltech.com>
…alkey

Signed-off-by: jonathanl-bq <72158117+jonathanl-bq@users.noreply.github.com>
@jonathanl-bq jonathanl-bq marked this pull request as ready for review September 10, 2024 22:08
@jonathanl-bq jonathanl-bq requested a review from a team as a code owner September 10, 2024 22:08


class RedisCluster:
class ValkeyCluster:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before python release, it was decided to leave this as it is

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? I don't really see how changing this to ValkeyCluster is problematic. This is a Valkey client.

acarbonetto and others added 4 commits September 10, 2024 17:18
…alkey

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Signed-off-by: Jonathan Louie <jonathanl@bitquilltech.com>
…alkey

Signed-off-by: jonathanl-bq <72158117+jonathanl-bq@users.noreply.github.com>
…alkey

Signed-off-by: jonathanl-bq <72158117+jonathanl-bq@users.noreply.github.com>
@jonathanl-bq jonathanl-bq merged commit ba7af10 into valkey-io:main Sep 11, 2024
@acarbonetto acarbonetto mentioned this pull request Sep 14, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python 🐍 Python wrapper

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants