Skip to content

Supports options.db to select redis db#2

Open
hyjin wants to merge 2 commits into
andyburke:masterfrom
hyjin:master
Open

Supports options.db to select redis db#2
hyjin wants to merge 2 commits into
andyburke:masterfrom
hyjin:master

Conversation

@hyjin

@hyjin hyjin commented Feb 19, 2014

Copy link
Copy Markdown

Hi, this change enables selecting db with options.db.

For example,

var emitter = new devents.EventEmitter({
    host: 'machine1.company.com',
    port: 6379,
    db: 1
});

@andyburke

Copy link
Copy Markdown
Owner

I'll accept this PR after a couple of cosmetic changes to conform to the rest of the code style:

  1. Spaces should be used with parens
  2. Braces for if statements should be on their own line

Example:

if ( condition )
{
    someFunction( someArgument );
}

@hyjin

hyjin commented Feb 19, 2014

Copy link
Copy Markdown
Author

@andyburke Here is update for code style. Thank you!

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.

2 participants