Page MenuHomePhabricator

Extend revert edges
AbandonedPublic

Authored by joshuaspence on Apr 30 2015, 9:56 PM.
Tags
None
Referenced Files
F19096150: D12634.diff
Thu, Dec 4, 8:08 AM
F19025089: D12634.diff
Mon, Nov 24, 5:29 AM
F18867974: D12634.id.diff
Nov 4 2025, 3:53 AM
F18867284: D12634.diff
Nov 3 2025, 10:53 PM
F18783296: D12634.diff
Oct 13 2025, 5:49 AM
F18749337: D12634.id33148.diff
Oct 4 2025, 3:25 AM
F18740513: D12634.id.diff
Oct 2 2025, 3:53 AM
F18736570: D12634.id.diff
Oct 1 2025, 6:43 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.