Merge lp:~allenap/maas/argparse-the-annoying--bug-1557434 into lp:~maas-committers/maas/trunk
Proposed by
Gavin Panella
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Gavin Panella | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 5173 | ||||
Proposed branch: | lp:~allenap/maas/argparse-the-annoying--bug-1557434 | ||||
Merge into: | lp:~maas-committers/maas/trunk | ||||
Diff against target: |
201 lines (+72/-17) 4 files modified
buildout.cfg (+21/-14) src/maascli/__init__.py (+8/-1) src/maascli/tests/test_cli.py (+1/-1) src/maascli/tests/test_integration.py (+42/-1) |
||||
To merge this branch: | bzr merge lp:~allenap/maas/argparse-the-annoying--bug-1557434 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mike Pontillo (community) | Approve | ||
Review via email: mp+299936@code.launchpad.net |
Commit message
For the MAAS CLI, mimic the error behaviour provided by argparse 1.1 on PyPI.
This differs substantially from the argparse 1.1 found in the standard library. This branch also forces the development environment to use argparse as found in the standard library instead of the one pulled in by unittest2.
To post a comment you must log in.
Just tested this locally. The behavior is *much* better. Many thanks for this.