Page MenuHomePhabricator

Consider identifier types when sorting clone URIs
ClosedPublic

Authored by epriestley on Jun 2 2016, 1:44 PM.
Tags
None
Referenced Files
F19521355: D16008.diff
Fri, Jan 16, 9:39 PM
F19516227: D16008.diff
Wed, Jan 14, 9:34 PM
F19508563: D16008.id38528.diff
Sat, Jan 10, 6:50 AM
F19491527: D16008.id38528.diff
Thu, Jan 8, 7:26 AM
F18907155: D16008.diff
Nov 8 2025, 10:11 PM
F18904619: D16008.id38528.diff
Nov 8 2025, 10:09 AM
F18902552: D16008.id.diff
Nov 8 2025, 1:44 AM
F18873056: D16008.diff
Nov 5 2025, 7:41 AM
Subscribers

Details

Summary

Fixes T11082. Currently, the /123/ and /CALLSIGN/ versions of the URI get the same score.

Also the scores are backwards.

Test Plan
  • Added getPublicCloneURI() output to repository listing.
  • Before patch, saw a repository with a callsign list a less-preferred ID-based URI.
  • After patch, saw the repository list the more-preferred callsign-based URI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Consider identifier types when sorting clone URIs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 2 2016, 1:56 PM
This revision was automatically updated to reflect the committed changes.