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

[dogshell][py3] print usage when no argument #123

Merged
merged 2 commits into from
Mar 11, 2016
Merged

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Mar 10, 2016

[dogshell][py3] print usage when no argument

In accordance with the behavior on Python 2.x, dog command with
Python 3.x should print the usage when no subparser is specified.
Instead, the uncalled code block is executed and raises:

AttributeError: 'Namespace' object has no attribute 'func'

Fix it and print usage.

More information: http://bugs.python.org/issue16308

Fix #85

[py3] better compatibility

Improve compatibility with Python 3.x.

@yannmh yannmh force-pushed the yann/dogshell-py3 branch from 05bd740 to 8e67720 Compare March 10, 2016 20:16
In accordance with the behavior on Python 2.x, `dog` command with
Python 3.x should print the usage when no subparser is specified.
Instead, the uncalled code block is executed and raises:
```
AttributeError: 'Namespace' object has no attribute 'func'
```
Fix it and print usage.

More information: http://bugs.python.org/issue16308

Fix #85
@yannmh yannmh force-pushed the yann/dogshell-py3 branch 2 times, most recently from 33accd9 to 62288c3 Compare March 10, 2016 20:54
Improve compatibility with Python 3.x.
@yannmh yannmh force-pushed the yann/dogshell-py3 branch from 62288c3 to f46d1b2 Compare March 10, 2016 22:49
@yannmh yannmh changed the title [test] support Python 3.5 [dogshell][py3] print usage when no argument Mar 10, 2016
@yannmh yannmh added this to the 0.11.0 milestone Mar 10, 2016
@yannmh
Copy link
Member Author

yannmh commented Mar 10, 2016

Another one for you @JohnLZeller 😄

@JohnLZeller
Copy link

👍 :) Don't forget to squash!

yannmh added a commit that referenced this pull request Mar 11, 2016
[dogshell][py3] print usage when no argument
@yannmh yannmh merged commit ff4a437 into master Mar 11, 2016
@yannmh yannmh deleted the yann/dogshell-py3 branch March 11, 2016 15:09
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