Merge lp:~gz/hydrazine/trivial_nameerror_find_status_877413 into lp:hydrazine
Proposed by
Martin Packman
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Martin Packman | ||||
Approved revision: | 100 | ||||
Merged at revision: | 100 | ||||
Proposed branch: | lp:~gz/hydrazine/trivial_nameerror_find_status_877413 | ||||
Merge into: | lp:hydrazine | ||||
Diff against target: |
15 lines (+2/-3) 1 file modified
feed-pqm (+2/-3) |
||||
To merge this branch: | bzr merge lp:~gz/hydrazine/trivial_nameerror_find_status_877413 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Riddell | Approve | ||
Review via email:
|
Description of the change
Correct a mistake from the fix for bug 703057, which factored filtering of merge proposals into a separate function. There was however, a use of the `find_status` at the bottom of the main loop that I missed:
<https:/
While here, I also removed the if-clause as it's always true as there are no break statements for the outer `while len(all_mps):` loop and it's therefore a confusing way of spelling `if True:`.
To post a comment you must log in.
This seems to result in it repeating the same merge proposal indefinately