Merge lp:~letterj/swift/getnodewhh into lp:~hudson-openstack/swift/trunk
Proposed by
Jay Payne
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | gholt | ||||
Approved revision: | 73 | ||||
Merged at revision: | 76 | ||||
Proposed branch: | lp:~letterj/swift/getnodewhh | ||||
Merge into: | lp:~hudson-openstack/swift/trunk | ||||
Diff against target: |
150 lines (+47/-91) 1 file modified
bin/swift-get-nodes (+47/-91) |
||||
To merge this branch: | bzr merge lp:~letterj/swift/getnodewhh | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chuck Thier (community) | Approve | ||
gholt (community) | Approve | ||
Review via email: mp+35624@code.launchpad.net |
Commit message
swift-get-nodes cleanup
To post a comment you must log in.
PEP8 Issues:
$ apt-get install pep8 get-nodes: 25:3: E111 indentation is not a multiple of four get-nodes: 43:21: E701 multiple statements on one line (colon) get-nodes: 43:27: E231 missing whitespace after ',' get-nodes: 69:68: W291 trailing whitespace
$ pep8 bin/swift-get-nodes
bin/swift-
bin/swift-
bin/swift-
bin/swift-