Homec4science

Purge loadRelativeEdges

Authored by Bob Trahan <bob.trahan@gmail.com> on Sep 13 2013, 20:40.

Description

Purge loadRelativeEdges

Summary:
Fixes T3821. Maybe. The existing code seemed to have a bug and actually return the commit phid. Judging by the function name this is not intended.

Also, sorry to step on toes here -- I thought no one was assigned and was curious about loadRelativeEdges and here we are...

Test Plan: lots of logic here as I have no idea how to use Releeph.

Reviewers: epriestley

Reviewed By: epriestley

CC: Korvin, aran

Maniphest Tasks: T3821

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

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Sep 13 2013, 20:40
Pushed
aubortJan 31 2017, 17:16
Parents
rPHc41c5933889e: Herald - make dry runs work for "apply once" rules after they have been applied
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPHea0dc5625d8f: Purge loadRelativeEdges (authored by Bob Trahan <bob.trahan@gmail.com>).Sep 13 2013, 20:40