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.

Sign up for GitHub

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

Jump to bottom

Add docs for config (issue #47) #50

Closed
wants to merge 2 commits into from
Closed

Add docs for config (issue #47) #50

wants to merge 2 commits into from
+35 −1

Conversation

philhofer
Copy link
Contributor

Added documentation for default config returned by NewConfig(), as well as max and min values. Fixes #47

@philhofer
Add docs for config
0541a35 
Added documentation for default config returned by NewConfig()
@mreiferson
Copy link
Member

Thanks @philhofer - do you mind annotating the ones that accept time as strings?

@philhofer
update config docs
cca7c3b 
Added description for time and boolean parsing/coercion.
@philhofer
Copy link
Contributor Author

@mreiferson I added a description of legal (coercible) values for durations and booleans.

@mreiferson
Copy link
Member

cool, brought this in via bitly/master@43ff0ca thanks!

@mreiferson mreiferson closed this Jun 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

opaque Config defaults
2 participants
@philhofer @mreiferson