Suggestion - Add gerrit code review entries to rietveld |
||||
Issue descriptionI have a few people who have migrated over to the new gerrit code review tool, but a majority of my code reviews are currently only on rietveld. Could we have it so one tool or the other shows all of the code reviews for both? Clicking on the link would obviously open the CL in the appropriate tool, but at least one unified list of the CLs I need to review would be great. Adding agable@ for triage or suggestions. Thanks! :)
,
Apr 7 2017
Yeah unfortunately this would be such a large body of effort in Rietveld that it simply isn't worth it. We could add a static link to the top of the Rietveld dashboard that says "see if you have reviews on gerrit here" or something, but making a bunch of API calls and formatting the result to look like the rest of the Rietveld dashboard is simply out of scope for a doomed product. Adding it to gerrit might be more feasible, but only as a plugin, and the dashboard page doesn't support plugins.
,
Apr 12 2017
Adding some folks who have done work on both gerrit and rietveld. Do you agree with my assessment that this, although awesome, is out of feasible scope?
,
Apr 12 2017
When we (Skia) dogfooded and then migrated completely to Gerrit we ran into the same problem, but it wasn't too bad. Had to check two systems for a while but notification emails made it manageable because it would take you to the right system when you click on the links in it. It is a good idea and if it was simple to do it should be done. But since it is quite complicated to implement I agree that it is out of feasible scope.
,
Apr 19 2017
,
May 1 2017
Unfortunately I believe that this is sufficiently complex to be infeasible on any time scale. I'm going to close this. If someone feels strongly otherwise, I'm happy to discuss and re-open but for now I think this isn't worth keeping around in our queues and dashboards. |
||||
►
Sign in to add a comment |
||||
Comment 1 by nyquist@chromium.org
, Apr 7 2017