Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for XGROUP container commands #1324

Merged
merged 25 commits into from
Jul 26, 2023

Conversation

vladvildanov
Copy link
Contributor

@vladvildanov vladvildanov commented Jun 19, 2023

Closes #1320
Closes #856
Closes #1321
Closes #1322
Closes #1323

@vladvildanov vladvildanov requested a review from a team June 19, 2023 13:01
@vladvildanov vladvildanov requested a review from tillkruss as a code owner June 19, 2023 13:01
@vladvildanov vladvildanov force-pushed the vv-1319-xgroup-commands branch from 09df345 to eaeaede Compare June 19, 2023 13:41
vladvildanov and others added 5 commits June 21, 2023 12:00
* Fixed bug with incorrect multiple words processing

* Convert subcommand string to lower case

* Update SubcommandStrategyResolver.php

* Added test coverage

* Codestyle fixes

---------

Co-authored-by: Till Krüss <[email protected]>
@coveralls
Copy link

coveralls commented Jun 21, 2023

Coverage Status

coverage: 77.759% (+0.09%) from 77.67% when pulling 747b35a on vladvildanov:vv-1319-xgroup-commands into 7c2e8e0 on predis:v2.x.

chayim
chayim previously approved these changes Jul 18, 2023
@vladvildanov vladvildanov changed the base branch from v2.x to v3.x July 26, 2023 12:53
@vladvildanov vladvildanov merged commit e600307 into predis:v3.x Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment