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

Redis v1 #70

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Redis v1 #70

wants to merge 11 commits into from

Conversation

maxtors
Copy link

@maxtors maxtors commented Jan 9, 2016

Added support to publish log entries on a redis channel. Not sure if everything is up to styling / convention, and probably some stuff needs work. Looking forward to feedback.

@thus
Copy link
Collaborator

thus commented Jan 10, 2016

Thanks for contributing!

I'll wait with any feedback, until you are done applying commits (8 yesterday, 3 new today). In the future, please wait until you are done coding before sending a pull request. It's kind of annoying when code is changing while reading it.

@maxtors
Copy link
Author

maxtors commented Jan 10, 2016

Sure, thats a good point. Sorry for that. Done with this round of changes (was going to commit todays changes to a new branch though). So ill open a new branch/PR with feedback and or discovered bugs / issues.

@thus
Copy link
Collaborator

thus commented Jan 10, 2016

Sounds good :) I'll take a look at your pull request as soon as I get the chance.

@lafeuil lafeuil mentioned this pull request Jan 11, 2016
@maxtors
Copy link
Author

maxtors commented Apr 14, 2016

Have you had a chance to take alook at this?

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.

None yet

2 participants