Skip to content

Don't assume redis_opts is enumerable#346

Merged
akira merged 1 commit intoakira:masterfrom
outstand:fix-redis-error-message
Oct 13, 2018
Merged

Don't assume redis_opts is enumerable#346
akira merged 1 commit intoakira:masterfrom
outstand:fix-redis-error-message

Conversation

@ryansch
Copy link
Contributor

@ryansch ryansch commented Oct 12, 2018

When the redis configuration is just the url string and there's a connection error, exq raises an exception instead of displaying the error message.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.254% when pulling 78d07c6 on outstand:fix-redis-error-message into 250e7d6 on akira:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.254% when pulling 78d07c6 on outstand:fix-redis-error-message into 250e7d6 on akira:master.

@coveralls
Copy link

coveralls commented Oct 12, 2018

Coverage Status

Coverage remained the same at 90.254% when pulling 7abbbc3 on outstand:fix-redis-error-message into 250e7d6 on akira:master.

@ryansch ryansch force-pushed the fix-redis-error-message branch from 78d07c6 to 7abbbc3 Compare October 12, 2018 16:46
@akira
Copy link
Owner

akira commented Oct 13, 2018

👍 thanks!

@akira akira merged commit c6c8501 into akira:master Oct 13, 2018
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