Merge lp://qastaging/~sergiusens/snapcraft/yaml_init into lp://qastaging/~snappy-dev/snapcraft/core
Proposed by
Sergio Schvezov
Status: | Merged |
---|---|
Approved by: | Sergio Schvezov |
Approved revision: | 166 |
Merged at revision: | 155 |
Proposed branch: | lp://qastaging/~sergiusens/snapcraft/yaml_init |
Merge into: | lp://qastaging/~snappy-dev/snapcraft/core |
Prerequisite: | lp://qastaging/~sergiusens/snapcraft/ubuntu-core |
Diff against target: |
444 lines (+132/-109) 3 files modified
snapcraft/cmds.py (+20/-3) snapcraft/tests/test_yaml.py (+20/-38) snapcraft/yaml.py (+92/-68) |
To merge this branch: | bzr merge lp://qastaging/~sergiusens/snapcraft/yaml_init |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Leo Arias (community) | Approve | ||
Review via email:
|
Commit message
Simplifying snapcraft.
Description of the change
The complexity is no longer 22 but 5 for __init__
To post a comment you must log in.
lgtm. Tests pass here.
I love that now not all the exceptions are sys exit. And thanks for using _ for variable names.