Page MenuHomePhabricator

Allow commits to be queried by repository using the `tagged(...)` typehaead function
ClosedPublic

Authored by epriestley on Feb 27 2017, 6:35 PM.
Tags
None
Referenced Files
F14415089: D17426.diff
Tue, Dec 24, 5:05 PM
Unknown Object (File)
Sun, Dec 22, 1:46 PM
Unknown Object (File)
Wed, Dec 18, 2:21 PM
Unknown Object (File)
Wed, Dec 18, 2:21 PM
Unknown Object (File)
Wed, Dec 18, 2:21 PM
Unknown Object (File)
Tue, Dec 17, 11:43 PM
Unknown Object (File)
Mon, Dec 16, 10:07 PM
Unknown Object (File)
Wed, Dec 11, 9:32 PM
Subscribers
None

Details

Summary

Fixes T12322. Allows you to search for commits using the tagged(...) repository function, so you can find "any commmit in any repository tagged with android" or similar.

I moved the function from Differential (which was the application using it) to Diffusion (which is more accurately the application which provides it).

I fixed a bug where searching for tagged(xyz) would have no effect (constraint was ignored) if there were no repositories tagged with "xyz". The fix isn't perfectly clean, but should work properly for the moment.

Test Plan
  • Searched with tagged(...) in Diffusion and Differential.
  • Searched by repository.
  • Searched with tagged(...) for a project with no tagged repositories.

Diff Detail

Repository
rP Phabricator
Branch
tagged1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15802
Build 20896: Run Core Tests
Build 20895: arc lint + arc unit