Merge lp:~ack/apt-btrfs-snapshot/fix-arg-parser into lp:apt-btrfs-snapshot
Proposed by
Alberto Donato
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~ack/apt-btrfs-snapshot/fix-arg-parser | ||||
Merge into: | lp:apt-btrfs-snapshot | ||||
Diff against target: |
54 lines (+2/-10) 1 file modified
apt-btrfs-snapshot (+2/-10) |
||||
To merge this branch: | bzr merge lp:~ack/apt-btrfs-snapshot/fix-arg-parser | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Vogt | Approve | ||
Review via email: mp+328776@code.launchpad.net |
Description of the change
This fixes the argparse setup not to error if no command is provided:
$ sudo apt-btrfs-snapshot
Traceback (most recent call last):
File "/usr/bin/
if args.command == "supported":
AttributeError: 'Namespace' object has no attribute 'command'
by setting the command parameter as required.
To post a comment you must log in.
Unmerged revisions
- 55. By Alberto Donato
-
Fix ArgumentParser setup.
Looks great, thanks for working on this!