Homestyx hydra
Diffusion hydra 35ccda922a31

Merge `diffusion.commitbranchesquery` into `diffusion.branchquery`

Description

Merge diffusion.commitbranchesquery into diffusion.branchquery

Summary:
Ref T4327. This is general cleanup since I was in this area of the code. Primarily, the Mercurial implementation here was completely broken and wrong:

  • It returned only one branch, but a commit can be present on many branches.
  • It did not account for multiple branch heads.
  • It returned a result implying the branch head pointed at the queried commit, which is no consistent or accurate.

Simplify the amount of API we're dealing with by collapsing this method into the very similar diffusion.branchquery method.

Test Plan: Looked at mercurial and git repositories and commits, branch information seemed correct.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4327

Differential Revision: https://secure.phabricator.com/D8003

Details

Provenance
epriestleyAuthored on Jan 17 2014, 7:11 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:4c2696120b81: Remove DiffusionBranchInformation in favor of DiffusionRepositoryRef
Branches
Unknown
Tags
Unknown

Event Timeline