Check for 'None' in argparse password #1

Merged
merged 1 commit into from Feb 5, 2013

Conversation

Projects
None yet
2 participants
@jbarratt

jbarratt commented Feb 5, 2013

The code as written wasn't working when I didn't supply a --password, it was trying to log in with password as None. It looks like the argparse object had the key for password, it just had a None value, so I added a check for that.

jackpal added a commit that referenced this pull request Feb 5, 2013

Merge pull request #1 from jbarratt/master
Check for 'None' in argparse password

@jackpal jackpal merged commit d1bba63 into jackpal:master Feb 5, 2013

@jackpal

This comment has been minimized.

Show comment Hide comment
@jackpal

jackpal Feb 5, 2013

Owner

Thanks!

Owner

jackpal commented Feb 5, 2013

Thanks!

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