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

fixes default option value (closes issue #26) #27

Merged
merged 1 commit into from
Mar 10, 2014
Merged

fixes default option value (closes issue #26) #27

merged 1 commit into from
Mar 10, 2014

Conversation

tjk
Copy link
Contributor

@tjk tjk commented Mar 8, 2014

See #26 for context, but after these changes, we honor default options.

$ ./example.out
Commands
    mine == false
    move == false
    create == false
    remove == false
    set == false
    ship == false
    shoot == false
Arguments
    x == (null)
    y == (null)
Flags
    --drifting == false
    --help == false
    --moored == false
    --version == false
Options
    --speed == 10

keleshev pushed a commit that referenced this pull request Mar 10, 2014
Merge pull request #27 from tjeezy/fix_default_options
7d85dc4 
fixes default option value (closes issue #26)
@keleshev keleshev merged commit 7d85dc4 into docopt:master Mar 10, 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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tjk @keleshev