Homec4science

Use first diff author for summary and test plan in commandeered revisions

Authored by vrana <jakubv@fb.com> on May 25 2012, 10:23.

Description

Use first diff author for summary and test plan in commandeered revisions

Summary:
I thought about it a little bit and this makes the most sense for me:

  1. Original author usually writes at least something and commander only updates it.
  2. There's a creation date of revision (= first diff) by these comments. I don't want to change this date because I use this information. Author should correspond to this date.
  3. It solves all our repros.

Test Plan: Display commandeered revision.

Reviewers: nh, epriestley

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D2577

Details

Committed
vrana <jakubv@fb.com>May 25 2012, 20:44
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf5e842ebd9af: dark console - introduce "request log" section
Branches
Unknown
Tags
Unknown

Event Timeline