Page MenuHomePhabricator

Move repository URI normalization out of PullLocalDaemon
ClosedPublic

Authored by epriestley on Jan 16 2014, 6:54 PM.
Tags
None
Referenced Files
F18938244: D7981.id18084.diff
Tue, Nov 11, 2:54 AM
F18904697: D7981.id.diff
Sat, Nov 8, 10:36 AM
F18886078: D7981.diff
Fri, Nov 7, 3:56 AM
F18855031: D7981.diff
Nov 1 2025, 2:20 AM
F18770063: D7981.id.diff
Oct 8 2025, 10:33 AM
F18764095: D7981.diff
Oct 7 2025, 4:15 AM
F18737609: D7981.diff
Oct 1 2025, 11:37 AM
F18734546: D7981.id.diff
Sep 30 2025, 10:56 PM
Subscribers

Details

Summary

Ref T4327. Moves us one small step forward toward testable change parsers by separating out this unrelated logic from the PullLocal daemon. We will also probably want to run this logic so we can do remote path lookups to limit the role of Arcanist Projects in the future, which is why I made the URI type (here, only "git") a parameter rather than calling this a GitURINormalizer or something.

Test Plan
  • Ran unit tests.
  • Ran repository discover on a correctly-configured remote repository.
  • Ran repository discover on an incorrectly-configured remote, got an error message.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped