sonyxperiadev / gerrit-events

MIT License
47 stars 62 forks source link

Deprecate get updated #52

Closed zaro0508 closed 8 years ago

zaro0508 commented 8 years ago

Change 3627295 prematurely added support to an unmerged gerrit stream events change[1]. That gerrit change started out with an updated property to indicate that an approval was updated however thru reviews that change ended up adding an oldValue property instead. This property is used to show the previous approval value when the score changed. Also the property is only appended to the stream event if the approval was changed so we can use this info to determine whether the approval was updated. We also deprecate getUpdate in favor of isUpdated.

[1] https://gerrit-review.googlesource.com/#/c/71051/