Skip to content

Improve phpredis support#32

Merged
willemstuursma merged 20 commits intophp-lock:masterfrom
TheLevti:develop
Apr 25, 2019
Merged

Improve phpredis support#32
willemstuursma merged 20 commits intophp-lock:masterfrom
TheLevti:develop

Conversation

@TheLevti
Copy link
Contributor

The phpredis extension has received many updates since support for it has been added to this library. Some of the features include the ability by the connection to be configured in such a way that all data that is going to redis will be serialized (none, php, igbinary) and/or compressed (none, lzf).

This pull request adds support for compression settings on the phpredis connection.

  • Adds support for phpreids compression.
  • Adds php7.3 to travis ci
  • Updated dev requirements & suggestions.
  • Updated suggested min version of the phpredis extension.
  • Updated tests to cover serialization and compression options of a phpredis connection.

@willemstuursma
Copy link
Contributor

Thanks @TheLevti. It looks very nice.

I'll review it in a few days. Feel free to ping me here if you think it is taking too long.

@TheLevti
Copy link
Contributor Author

No problem. This is not urgent. Just noticed that issue when I was playing around with phpredis.

@willemstuursma willemstuursma merged commit a7fa877 into php-lock:master Apr 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants