Merge lp://qastaging/~dobey/tarmac/votes-trustedreviewer into lp://qastaging/tarmac

Proposed by dobey
Status: Merged
Merged at revision: 370
Proposed branch: lp://qastaging/~dobey/tarmac/votes-trustedreviewer
Merge into: lp://qastaging/tarmac
Diff against target: 242 lines (+67/-46)
6 files modified
tarmac/bin/commands.py (+10/-11)
tarmac/branch.py (+6/-6)
tarmac/plugins/tests/test_votes.py (+18/-5)
tarmac/plugins/votes.py (+6/-3)
tarmac/tests/test_branch.py (+5/-4)
tarmac/tests/test_commands.py (+22/-17)
To merge this branch: bzr merge lp://qastaging/~dobey/tarmac/votes-trustedreviewer
Reviewer Review Type Date Requested Status
Paul Hummer Pending
Review via email: mp+37992@code.qastaging.launchpad.net

Commit message

Move the isPersonTrustedReviewer check into the votes plug-in
Store all reviewers and votes in the reviews revprop
Stop storing just a list of approved reviews in the reviewers revprop

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches