Skip to content

Support for redis 4#101

Open
amatriain wants to merge 2 commits into
steelThread:masterfrom
amatriain:fix-redis-connect
Open

Support for redis 4#101
amatriain wants to merge 2 commits into
steelThread:masterfrom
amatriain:fix-redis-connect

Conversation

@amatriain

Copy link
Copy Markdown
Contributor

Since redis 4 the previously deprecated #connect method has been removed. Redmon should use #new instead.

Since redis 4 the previously deprecated #connect method has been removed. Redmon should use #new instead.
@amatriain

Copy link
Copy Markdown
Contributor Author

Fixes #99

@coveralls

coveralls commented Dec 20, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-3.3%) to 78.947% when pulling 958a1d4 on amatriain:fix-redis-connect into 7f567f7 on steelThread:master.

@amatriain

amatriain commented Dec 20, 2017

Copy link
Copy Markdown
Contributor Author

Travis errors seem caused by incompatibility between ruby versions and some gems? I'm pretty sure the code changes in this PR are not causing the errors I'm seeing in Travis logs.

@LuisErnestoZamb

Copy link
Copy Markdown

Hi, When this is going to be merged?

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.

3 participants