Remove implicit sorting from "MarkerRefQuery"
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 29, 4:02 PM

	Unknown Object (File)
	Sat, Apr 27, 9:07 AM

	Unknown Object (File)
	Fri, Apr 26, 8:54 PM

	Unknown Object (File)
	Wed, Apr 24, 11:52 PM

	Unknown Object (File)
	Sat, Apr 20, 5:22 PM

	Unknown Object (File)
	Fri, Apr 19, 4:56 PM

	Unknown Object (File)
	Thu, Apr 11, 8:08 AM

	Unknown Object (File)
	Apr 3 2024, 2:04 PM

View All 90 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

Branch

landx12

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 24671
Build 34019: Run Core Tests
Build 34018: arc lint + arc unit

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 50852

View Options

src/repository/marker/ArcanistRepositoryMarkerQuery.php

Remove implicit sorting from "MarkerRefQuery"ClosedPublicActions