Page MenuHomePhabricator

Extend revert edges
AbandonedPublic

Authored by joshuaspence on Apr 30 2015, 9:56 PM.
Tags
None
Referenced Files
F13102982: D12634.diff
Sat, Apr 27, 7:40 AM
Unknown Object (File)
Thu, Apr 25, 2:46 AM
Unknown Object (File)
Fri, Apr 19, 2:17 AM
Unknown Object (File)
Fri, Apr 19, 2:17 AM
Unknown Object (File)
Fri, Apr 19, 2:17 AM
Unknown Object (File)
Fri, Apr 19, 2:17 AM
Unknown Object (File)
Fri, Apr 19, 2:17 AM
Unknown Object (File)
Fri, Apr 19, 2:17 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.