Page MenuHomePhabricator
Create Task
Maniphest T229513

Define tools/processes to help manage code reviews
Open, Needs TriagePublic
Actions

Description

Building on what we learned during our Code Review work group discussions, we need to better define how we can support/enable the code review activity by providing the tools and process for teams to use to manage the code review workload.

@Krinkle (Gerrit Dashboards) and @WDoranWMF (Phabricator) spoke to this a bit during our 2019.07.23 session.

In addition to tooling and process, we need to figure out a way to make this available to more teams/people, so a "roll out" of sorts will need to be supported.

Related Objects

Event Timeline

Jrbranaa created this task.Jul 31 2019, 11:17 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 31 2019, 11:17 PM
WDoranWMF added a subscriber: cicalese.Aug 1 2019, 8:14 PM

In CPT we've started experimenting with SMW as a window into both gerrit and phab, we're still playing with it but @cicalese has built two interesting views:
Open Gerrit changes for Active Tasks
and
Task Creation vs Closure

CCicalese_WMF edited subscribers, added: CCicalese_WMF; removed: cicalese.Aug 2 2019, 2:30 AM
Tgr added a subscriber: Tgr.Aug 5 2019, 7:23 AM

Somewhat related (but more about helping new developers than experienced ones): T214397: Gerrit: reviewers-by-blame plugin improvements

Aklapper added a comment.Aug 5 2019, 11:11 PM

FYI, current docs about setting up Gerrit dashboards are at https://www.mediawiki.org/wiki/Gerrit/Advanced_usage#Gerrit_project_dashboard

Krinkle added a comment.Aug 5 2019, 11:38 PM
In T229513#5394917, @Aklapper wrote:

FYI, current docs about setting up Gerrit dashboards are at https://www.mediawiki.org/wiki/Gerrit/Advanced_usage#Gerrit_project_dashboard

What he said :) This section reflect the process as it was used by Editing team, and has been updated by me to reflect how we still use it in the Performance team.

Jrbranaa moved this task from Backlog to In-Progress on the Code-Review-Workgroup board.Aug 6 2019, 2:00 PM
Aklapper renamed this task from Define tools/processes to help manage reviews to Define tools/processes to help manage code reviews.Aug 9 2019, 11:29 AM
mmodell added a subscriber: mmodell.Sep 10 2019, 2:05 PM
DannyS712 added a subscriber: DannyS712.Feb 14 2020, 1:20 AM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL