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

minor changes #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BFRS02064-vedant
Copy link

No description provided.

@BFRS02064-vedant
Copy link
Author

Adding phpredis diver support for redis store

@ackintosh
Copy link
Owner

@BFRS02064-vedant Thank you for this PR.

Have you tested this with Illuminate\Redis\Connections\PhpRedisConnection? If so, was it successful?

@BFRS02064-vedant
Copy link
Author

Yes I have tested with Illuminate\Redis\Connections\PhpRedisConnection

@ackintosh
Copy link
Owner

Cool. Could you please add a test to verify this change? The following existing tests might be useful as references:

If you need any assistance or have questions, feel free to ask. Thanks for your contribution. 🙂

@ackintosh
Copy link
Owner

I have made and merged fixes for the CI issue. Could you please merge the latest master branch at your earliest convinience?

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