Page MenuHomePhabricator

Deprecate Conduit method "diffusion.getrecentcommitsbypath"
ClosedPublic

Authored by epriestley on Aug 2 2018, 11:39 PM.
Tags
None
Referenced Files
F18777981: D19559.diff
Sat, Oct 11, 7:04 AM
F18704725: D19559.id46763.diff
Sun, Sep 28, 8:34 AM
F18701183: D19559.diff
Sat, Sep 27, 5:49 PM
F18657428: D19559.diff
Sep 23 2025, 1:49 AM
F18628917: D19559.id.diff
Sep 16 2025, 7:34 AM
F18611393: D19559.id46762.diff
Sep 14 2025, 7:09 AM
F18509721: D19559.id.diff
Sep 5 2025, 3:34 AM
F18503070: D19559.diff
Sep 4 2025, 10:50 PM
Subscribers
Tokens
"Like" token, awarded by artms.

Details

Summary

See D19558. This method has no callers and just wraps diffusion.historyquery, since D5960 (2013).

This was introduced in D315 (which didn't make it out of FB, I think) inside Facebook for unclear purposes in 2011.

Test Plan

Grepped for callers, found none.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Aug 2 2018, 11:58 PM

(You probably have years until this disappears from the upstream since it's already implemented in terms of diffusion.historyquery and we don't generally remove methods without some specific reason, but if you have opportunities to migrate toward diffusion.historyquery it couldn't hurt.)

This revision was automatically updated to reflect the committed changes.