Skip to content

Conversation

@dwighthubbard
Copy link

No description provided.

@prologic
Copy link

prologic commented Aug 2, 2015

👍 I'm defintely going to use this :)

@prologic
Copy link

Ignore my last silly comment :-) I'm merging your fork into mine and testing this out

@prologic
Copy link

So 👍 it works really nicely :)

@prologic
Copy link

prologic commented Sep 1, 2015

Hey @dwighthubbard I tried out your patch/pull-request in my branch and I found an issue that I'd like to address:

  • We need a way to be explicit about whether we use redis or redislite here.

Right now I think you use Redislite if it's "importable". The use-case I want to have for https://github.com/prologic/charla is:

charla --embedded

Makes sense? :)

@dwighthubbard
Copy link
Author

Redislite will pass the request through to the Redis module if the user passes a hostname or socketfile of a Redis server as an argument. This was done in order to prevent the need to write code in every app to fail back to Redis if the user doesn't want to use the embedded server.

kiddouk added a commit that referenced this pull request Feb 7, 2016
Add support to optionally use redislite
@kiddouk kiddouk merged commit 39044f1 into kiddouk:master Feb 7, 2016
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