Skip to content

Conversation

@vladvildanov
Copy link
Contributor

vladvildanov and others added 30 commits January 25, 2023 07:59
* Added support for SORT_RO command

* Codestyle fixes

* Added command description

---------

Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local>
* Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL

* Changed ContainerInterface and AbstractContainer

* Re-implement logic of abstract methods

---------

Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local>
Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local>
…redis#1168)

* Add option for disabling reads to slaves in replication topologies

(cherry picked from commit e132c31)

* remove empty line

---------

Co-authored-by: Till Krüss <tillkruss@users.noreply.github.com>
* match `MasterSlaveReplication`

also the name of this method 😬

* Update SentinelReplication.php
* Added CODEOWNERS file

* Update CODEOWNERS

* Applied whole Redis commands path as @vladvildanov codeowner

* Set @predis/team as Redis commands codeowners

* Update CODEOWNERS

---------

Co-authored-by: Vladyslav Vildanov <vladyslavvildanov@Vladyslav-Vildanov-MacBook-Pro.local>
Co-authored-by: Till Krüss <tillkruss@users.noreply.github.com>
* Deprecate webdis and phpiredis

* remove space
@vladvildanov vladvildanov requested review from a team, chayim and tillkruss April 3, 2023 09:27
Copy link
Contributor

@shacharPash shacharPash left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@chayim chayim left a comment

Choose a reason for hiding this comment

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

Let's wait on @tillkruss to merge obvi

@vladvildanov
Copy link
Contributor Author

@chayim It's not a main PR, it's just resolving conflicts between v2.x and main. After this one will be merged I can create a release PR

@vladvildanov vladvildanov merged commit af34718 into predis:main Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

7 participants