Page MenuHomePhabricator
Create Task
Maniphest T114311

[keyresult] Code review RFC (Differential) - Write, publish, publicize, and respond to/incorporate feedback.
Closed, ResolvedPublic
Actions

Description

Quarterly goal tracking task.

Subtasks (where work/discussion will happen) will be created and/or marked as blockers of this one (but limited in topic to the RFC).

Related Objects
Search...

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.
StatusSubtypeAssignedTask
· · ·
ResolveddemonT114311 [keyresult] Code review RFC (Differential) - Write, publish, publicize, and respond to/incorporate feedback.
ResolvedmmodellT117058 Initial documentation of example Differential workflows (with Gerrit equivalents)
· · ·

Event Timeline

greg created this task.Sep 30 2015, 9:21 PM
greg raised the priority of this task from to Medium.
greg updated the task description. (Show Details)
greg added a project: releng-201516-q2.
greg added subscribers: greg, demon, mmodell.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 30 2015, 9:21 PM
greg mentioned this in T114320: Code-review migration to Differential status/discussion.Sep 30 2015, 10:29 PM
saper added a subscriber: saper.Oct 19 2015, 11:39 AM
Aklapper mentioned this in T101686: Goal: Define potential actions to reduce code review queues and waiting times.Oct 22 2015, 4:23 PM

Rather a note to myself, but also for general awareness:
Any potential change on the technical side (our review infrastructure) needs to be taken into account in T101686: Goal: Define potential actions to reduce code review queues and waiting times.

Aklapper mentioned this in T114419: Event on "Make code review not suck".Oct 23 2015, 10:09 PM
Elitre added a subscriber: Elitre.Oct 26 2015, 12:09 PM
Ricordisamoa added a subscriber: Ricordisamoa.Oct 26 2015, 1:41 PM
Jay8g added a subscriber: Jay8g.Oct 31 2015, 4:27 AM
Aklapper added a comment.Nov 14 2015, 5:11 PM

@greg: Should this task block / be listed in T584: Define main tasks (epics) for code review in Phabricator?

• chasemp added a subscriber: • chasemp.Nov 30 2015, 8:49 PM
demon added a comment.Nov 30 2015, 9:51 PM

RfC filed: T114311. Needs refinement & discussion now.

demon added a comment.Nov 30 2015, 9:53 PM

Wrong link, see T119908: [RfC]: Migrate code review / management from Gerrit to Phabricator

greg added a comment.Dec 1 2015, 4:46 PM
In T114311#1805691, @Aklapper wrote:

@greg: Should this task block / be listed in T584: Define main tasks (epics) for code review in Phabricator?

I'm still failing to see the benefit of that task in our current situation. I'll take a stab at cleaning it up later this week.

Liuxinyu970226 added a subscriber: Liuxinyu970226.Dec 5 2015, 2:22 AM
Aklapper mentioned this in T115844: Have a second Gerrit (Code Review backlog) cleanup day.Dec 29 2015, 6:07 PM
ArielGlenn added a subscriber: ArielGlenn.Jan 4 2016, 11:19 PM
greg closed this task as Resolved.Jan 20 2016, 12:22 AM
greg claimed this task.

This step (Q2 quarterly goal) is done, the work on the actual RFC is at T119908.

Restricted Application added a project: User-greg. · View Herald TranscriptJan 20 2016, 12:22 AM
greg reassigned this task from greg to demon.Jan 20 2016, 12:23 AM
greg removed a project: User-greg.
greg set Security to None.
Liuxinyu970226 removed a subscriber: Liuxinyu970226.Jan 20 2016, 2:26 AM
greg closed subtask T117058: Initial documentation of example Differential workflows (with Gerrit equivalents) as Resolved.Jul 1 2016, 8:49 PM
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