Page MenuHomePhabricator

Remove old "Landing Strategy" code
ClosedPublic

Authored by epriestley on Jun 23 2017, 3:10 PM.
Tags
None
Referenced Files
F15574264: D18150.diff
Tue, May 6, 12:31 AM
F15568117: D18150.id43670.diff
Fri, May 2, 8:32 PM
F15564870: D18150.diff
Wed, Apr 30, 4:46 PM
F15541650: D18150.id43669.diff
Fri, Apr 25, 12:45 PM
F15513045: D18150.id.diff
Thu, Apr 17, 4:04 PM
F15513030: D18150.diff
Thu, Apr 17, 3:51 PM
F15463144: D18150.id43670.diff
Apr 1 2025, 8:43 PM
F15462273: D18150.id43669.diff
Apr 1 2025, 1:36 PM
Subscribers
None

Details

Summary

Fixes T12869. This is a very old, pre-Drydock chunk of code from D7486 and some followups.

It does three things:

  • "Land to Hosted Git": Obsoleted by Drydock, has been commented out in HEAD for a very long time with no complaints. Disabled by D8719 in 2014.
  • "Land to Hosted Mercurial": Could be obsoleted by Drydock with a fairly small amount of work, but currently has no replacement. Unclear if this sees any real use. Not actually disabled at HEAD.
  • "Land to GitHub": Use GitHub OAuth credentials to land to GitHub. This is sort of theoretically useful and has no analog today. Disabled by D13022 in 2015.

This stuff was largely disabled a long time ago and we haven't seen users hitting issues with it. This could all be moved to an extension today if anyone still relies on it.

Test Plan

Grepped for removed classes, browsed Differential.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable