Clarify that options takes a database parameter #145

Merged
merged 1 commit into from Apr 17, 2014

Conversation

Projects
None yet
2 participants
Contributor

ianlyons commented Apr 13, 2014

Hi,

First, many thanks for putting this together.

I just got (stupidly) stuck on this for a bit until I read the docs a bit more closely and realized that the DB name is part of the options object. I figured I would add a couple of lines to this to clarify even further for developers down the road. Many thanks!

@ianlyons ianlyons Clarify that options takes a database parameter
Hi,

First, many thanks for putting this together. 

I just got (stupidly) stuck on this for a bit until I read the docs a bit more closely and realized that the DB name is part of the options object. I figured I would add a couple of lines to this to clarify even further for developers down the road. Many thanks!
3c10931

@patriksimek patriksimek added a commit that referenced this pull request Apr 17, 2014

@patriksimek patriksimek Merge pull request #145 from ianlyons/patch-1
Clarify that options takes a database parameter
84d7b00

@patriksimek patriksimek merged commit 84d7b00 into tediousjs:master Apr 17, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Collaborator

patriksimek commented Apr 17, 2014

Thanks.

ianlyons deleted the ianlyons:patch-1 branch Apr 17, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment