diff options
author | Jenkins <jenkins@review.openstack.org> | 2014-08-16 00:40:24 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2014-08-16 00:40:24 +0000 |
commit | 9a95e715eca6d483e7531c5cb47cfc44bf171ba9 (patch) | |
tree | 0297e9f778021d9475cf5dd100f80216eb95df9e | |
parent | e2259ea455273cdb71e86753cda776f3054ecc06 (diff) | |
parent | f3348a7620eab97ee473f53321206a5ccc158202 (diff) | |
download | zuul-9a95e715eca6d483e7531c5cb47cfc44bf171ba9.tar.gz |
Merge "Update gerrit change attributes even if merged"
-rw-r--r-- | zuul/trigger/gerrit.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/zuul/trigger/gerrit.py b/zuul/trigger/gerrit.py index 754c94270..4d4deb8b6 100644 --- a/zuul/trigger/gerrit.py +++ b/zuul/trigger/gerrit.py @@ -364,6 +364,10 @@ class Gerrit(object): change.is_current_patchset = False change.is_merged = self._isMerged(change) + change.approvals = data['currentPatchSet'].get('approvals', []) + change.open = data['open'] + change.status = data['status'] + if change.is_merged: # This change is merged, so we don't need to look any further # for dependencies. @@ -386,10 +390,6 @@ class Gerrit(object): if not dep.is_merged and dep.is_current_patchset: change.needed_by_changes.append(dep) - change.approvals = data['currentPatchSet'].get('approvals', []) - change.open = data['open'] - change.status = data['status'] - return change def getGitUrl(self, project): |