please add gerrit support to my_reviews.py |
|||
Issue descriptionit appears to only support rietveld.
,
Sep 6 2017
my_activity.py shows Gerrit reviews
,
Sep 6 2017
should we delete it then?
,
Sep 6 2017
It's on my list of things to delete after Rietveld is 100% read-only. I'll probably wait a quarter to do so in case people want to use it for perf/okrs.
,
Sep 14 2017
Doesn't work for me. It even says in the output "Get rietveld stats about the review you done, or forgot to do." Am I using it wrong?
,
Sep 14 2017
,
Sep 14 2017
NM. I finally realized that you're saying my_activity lists Gerrit, as opposed to my_reviews. At least when I used the two scripts, they had slightly different capabilities. So before we throw one away, we might want to figure out what those differences were, and if we still need them. (I used them in a script that's somewhat popular around perf time, for summaries - but can't test right now, since I'm OOO) |
|||
►
Sign in to add a comment |
|||
Comment 1 by zhangtiff@chromium.org
, Aug 31 2017Components: -Infra Infra>Git