Page MenuHomePhabricator

Move repository URI normalization out of PullLocalDaemon
ClosedPublic

Authored by epriestley on Jan 16 2014, 6:54 PM.
Tags
None
Referenced Files
F15589419: D7981.id.diff
Fri, May 9, 1:30 PM
F15536605: D7981.id18057.diff
Thu, Apr 24, 10:47 AM
F15528795: D7981.id18084.diff
Tue, Apr 22, 2:03 PM
F15528352: D7981.id.diff
Tue, Apr 22, 10:38 AM
F15523866: D7981.diff
Mon, Apr 21, 4:15 AM
F15509549: D7981.id18084.diff
Wed, Apr 16, 1:50 PM
F15485845: D7981.diff
Thu, Apr 10, 2:58 AM
F15437935: D7981.diff
Mar 25 2025, 10:14 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