Remove implicit sorting from "MarkerRefQuery"
ClosedPublic
Actions

Authored by epriestley on Jun 17 2020, 5:31 PM.

Tags

None

Referenced Files

	F18855261: D21364.id50890.diff
	Sat, Nov 1, 4:37 AM

	F18825642: D21364.diff
	Thu, Oct 23, 11:14 PM

	F18772114: D21364.id50890.diff
	Thu, Oct 9, 12:25 AM

	F18737632: D21364.diff
	Oct 1 2025, 11:42 AM

	F18671791: D21364.id.diff
	Sep 25 2025, 5:18 AM

	F18630834: D21364.diff
	Sep 16 2025, 1:18 PM

	F18198677: D21364.diff
	Aug 18 2025, 1:09 AM

	F18104302: D21364.id50852.diff
	Aug 10 2025, 11:18 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13546: Modernize the "arc land" workflow
Commits: rARC10c4a551ae9d: Remove implicit sorting from "MarkerRefQuery"

Summary

Ref T13546. This is no longer necessary after the introduction of "msortv_natural()", which can handle natural string sorting.

Test Plan

Ran "arc branches", saw the same sorting applied.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 17 2020, 5:31 PM

Harbormaster completed remote builds in B24671: Diff 50852.Jun 17 2020, 5:31 PM

epriestley requested review of this revision.Jun 17 2020, 5:31 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 30 2020, 10:50 PM

Closed by commit rARC10c4a551ae9d: Remove implicit sorting from "MarkerRefQuery" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC10c4a551ae9d: Remove implicit sorting from "MarkerRefQuery".

Revision Contents
Changeset List

Path

Size

src/

repository/

marker/

ArcanistRepositoryMarkerQuery.php

15 lines

Diff 50890

View Options

src/repository/marker/ArcanistRepositoryMarkerQuery.php

Remove implicit sorting from "MarkerRefQuery"ClosedPublicActions