Page MenuHomePhabricator

Policy - clean up the deprecated diffusion.getcommits
ClosedPublic

Authored by btrahan on Jan 30 2015, 7:46 PM.
Tags
None
Referenced Files
F12846034: D11581.id.diff
Fri, Mar 29, 12:41 AM
Unknown Object (File)
Mon, Mar 25, 11:44 AM
Unknown Object (File)
Mon, Mar 25, 11:44 AM
Unknown Object (File)
Mon, Mar 25, 11:44 AM
Unknown Object (File)
Mon, Mar 25, 11:44 AM
Unknown Object (File)
Tue, Mar 5, 8:01 AM
Unknown Object (File)
Feb 23 2024, 12:19 PM
Unknown Object (File)
Feb 23 2024, 12:19 PM
Subscribers

Details

Summary

Ref T7094. Could just delete this end point too I guess? Needed to add "withCommitPHIDs" to the differentialrevisionquery to get this done.

Test Plan

used diffusion.getcommits from conduit console and got a sensible result for a query for two commits, one with a diff and one without.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Policy - clean up the deprecated diffusion.getcommits.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

I think this approach is desirable in the long run, since this capability is likely useful elsewhere even once we eventually nuke this method.

This revision is now accepted and ready to land.Jan 30 2015, 7:50 PM
This revision was automatically updated to reflect the committed changes.