Page MenuHomePhabricator
Differential D20605

Fix two "msort()" vs "msortv()" issues in "arc land"
ClosedPublic
Actions

Authored by epriestley on Jun 20 2019, 9:41 PM.
Tags
None
Referenced Files
F14406399: D20605.diff
Mon, Dec 23, 2:02 PM
Unknown Object (File)
Wed, Dec 18, 4:04 PM
Unknown Object (File)
Wed, Dec 18, 1:32 AM
Unknown Object (File)
Mon, Dec 16, 10:25 AM
Unknown Object (File)
Fri, Dec 13, 1:09 PM
Unknown Object (File)
Fri, Dec 13, 12:35 PM
Unknown Object (File)
Sat, Dec 7, 5:21 PM
Unknown Object (File)
Fri, Dec 6, 12:53 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13303: There is an extra or missing "array_reverse()" somewhere after recent transaction strength sorting changes
Commits
rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land"
Summary

See https://github.com/phacility/arcanist/pull/242. Ref T13303. A little more fallout turned up in arc.

Test Plan

grep msort(

Diff Detail

Repository
rARC Arcanist
Branch
msort1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23013
Build 31595: Run Core Tests
Build 31594: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
workflow/
4 lines

Diff 49141

View Options

src/workflow/ArcanistLandWorkflow.php

Loading...
Phabricator · Built by Phacility