Page MenuHomePhabricator

Extend revert edges
AbandonedPublic

Authored by joshuaspence on Apr 30 2015, 9:56 PM.
Tags
None
Referenced Files
F18208697: D12634.diff
Mon, Aug 18, 9:39 PM
F18160093: D12634.id33148.diff
Fri, Aug 15, 1:37 AM
F18101943: D12634.id30394.diff
Sat, Aug 9, 4:55 PM
F18095442: D12634.id30332.diff
Thu, Aug 7, 11:37 PM
F18089005: D12634.id30357.diff
Wed, Aug 6, 10:14 AM
F18002877: D12634.id30385.diff
Sat, Aug 2, 11:09 AM
F17825138: D12634.id.diff
Sat, Jul 26, 6:06 AM
F17804645: D12634.diff
Fri, Jul 25, 11:39 AM

Details

Summary

Ref T1751. Extends revert edges by adding new edge types:

  • DifferentialRevisionRevertedByCommitEdgeType
  • DiffusionCommitRevertsRevisionEdgeType
  • DiffusionCommitRevertsTasksEdgeType
  • ManiphestTaskRevertedByCommitEdgeType

This means that when a commit is reverted, an edge (and transaction) is created on any tasks/revisions which are associated with the reverted commit(s).

Test Plan

Reverted a commit which contained Ref Tx and Fixes Ty and also mentioned Tz. Saw reverting edges created for Tx and Ty.

Event Timeline

joshuaspence retitled this revision from to Extend revert edges.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited edge metadata.

Seems to work from some initial testing

Hmm... this doesn't work initially, but does work when I run ./bin/repository reparse.

joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

One thing that would be nice is to disable the "mention" for reverts:

ment.png (77×1 px, 19 KB)

Not actively working on this.

This may mostly have been covered by D20469, although that change does not extend to reverting/reopening tasks.