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

[5.8] Add Redis 5 support #29606

Merged
merged 3 commits into from Aug 16, 2019

Conversation

@driesvints
Copy link
Member

commented Aug 16, 2019

These changes should be backwards compatible. I had to differentiate the tests between the PhpRedis and Predis connections because Predis doesn't seems to handle the 5.0 changes too well atm. And it doesn't seems to be maintained anymore...

Fixes #29029

driesvints added 3 commits Aug 16, 2019
@driesvints driesvints referenced this pull request Aug 16, 2019

@taylorotwell taylorotwell merged commit 66b3188 into laravel:5.8 Aug 16, 2019

2 checks passed

continuous-integration/styleci/pr The analysis has passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tillkruss

This comment has been minimized.

Copy link
Member

commented Aug 20, 2019

Yikes, no commits for two years.

nrk/predis#524

@driesvints

This comment has been minimized.

Copy link
Member Author

commented Aug 20, 2019

@driesvints driesvints deleted the driesvints:redis-5 branch Aug 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.