Page MenuHomePhabricator

Move repository URI normalization out of PullLocalDaemon
ClosedPublic

Authored by epriestley on Jan 16 2014, 6:54 PM.
Tags
None
Referenced Files
F14093174: D7981.diff
Mon, Nov 25, 9:57 AM
Unknown Object (File)
Thu, Nov 21, 2:42 PM
Unknown Object (File)
Tue, Nov 19, 3:20 AM
Unknown Object (File)
Sun, Nov 17, 7:21 AM
Unknown Object (File)
Wed, Nov 6, 12:36 AM
Unknown Object (File)
Wed, Nov 6, 12:36 AM
Unknown Object (File)
Wed, Nov 6, 12:36 AM
Unknown Object (File)
Mon, Nov 4, 10:52 AM
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