Skip to content

chore: remove a redundant method from Ring, Client and ClusterClient#3401

Merged
ndyakov merged 2 commits into
redis:masterfrom
cxljs:chore/remove-redundant-code
Jun 16, 2025
Merged

chore: remove a redundant method from Ring, Client and ClusterClient#3401
ndyakov merged 2 commits into
redis:masterfrom
cxljs:chore/remove-redundant-code

Conversation

@cxljs

@cxljs cxljs commented Jun 10, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: fukua95 <fukua95@gmail.com>
@cxljs cxljs force-pushed the chore/remove-redundant-code branch from 5c7c133 to 983fb6c Compare June 10, 2025 12:01
@cxljs cxljs changed the title chore: remove a redundant method from Ring chore: remove a redundant method from Ring, Client and ClusterClient Jun 10, 2025
@ndyakov

ndyakov commented Jun 10, 2025

Copy link
Copy Markdown
Member

@fukua95, the tests should work with the baseClient anyway, let's keep them.

@cxljs

cxljs commented Jun 12, 2025

Copy link
Copy Markdown
Contributor Author

Got it.

@ndyakov ndyakov merged commit 82b00cc into redis:master Jun 16, 2025
16 checks passed
ofekshenawa pushed a commit to ofekshenawa/go-redis that referenced this pull request Jun 30, 2025
Signed-off-by: fukua95 <fukua95@gmail.com>
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com>
ofekshenawa pushed a commit that referenced this pull request Aug 10, 2025
Signed-off-by: fukua95 <fukua95@gmail.com>
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants