Page MenuHomePhabricator

Move more discovery responsibilities into DiscoveryEngine
ClosedPublic

Authored by epriestley on Jan 17 2014, 9:27 PM.
Tags
None
Referenced Files
F18848765: D7997.id.diff
Oct 30 2025, 6:12 AM
F18844798: D7997.diff
Oct 29 2025, 7:40 AM
F18729767: D7997.diff
Sep 30 2025, 11:50 AM
F18637469: D7997.diff
Sep 17 2025, 7:39 AM
F18591464: D7997.id18106.diff
Sep 12 2025, 9:18 AM
F18590618: D7997.id18106.diff
Sep 12 2025, 6:34 AM
F18590033: D7997.diff
Sep 12 2025, 4:59 AM
F18586693: D7997.id18095.diff
Sep 11 2025, 7:25 PM
Subscribers

Details

Summary

Ref T4327. This moves the last pieces of discovery responsibility out of the PullLocal daemon and into the DiscoveryEngine.

(This makes it easier to discover repositories in unit tests in the future, since we don't need to build a PullLocal daemon and can just build a DiscoveryEngine.)

Test Plan

Ran phd debug pulllocal. Ran repostory discover.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php
61

This indirectly fixes a small potential bug, where a commit could get marked in the cache but not actually recorded.