Merge lp:~dpb/juju-deployer/empty-options-1361883 into lp:juju-deployer
Proposed by
David Britton
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 121 | ||||
Proposed branch: | lp:~dpb/juju-deployer/empty-options-1361883 | ||||
Merge into: | lp:juju-deployer | ||||
Diff against target: |
72 lines (+35/-1) 5 files modified
Makefile (+2/-0) deployer/deployment.py (+4/-1) deployer/tests/test_data/negative.cfg (+10/-0) deployer/tests/test_data/negative.yaml (+5/-0) deployer/tests/test_deployment.py (+14/-0) |
||||
To merge this branch: | bzr merge lp:~dpb/juju-deployer/empty-options-1361883 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Kapil Thangavelu | Approve | ||
Review via email: mp+232332@code.launchpad.net |
Description of the change
Simple none type check to fix. test cases fail if you revert the patch to deployment.py
To post a comment you must log in.
thanks, fwiw for test running i typically just do python setup.py test