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
Unknown Object (File)
Thu, Apr 25, 4:33 PM
Unknown Object (File)
Thu, Apr 25, 12:32 AM
Unknown Object (File)
Sat, Apr 20, 4:51 PM
Unknown Object (File)
Sun, Apr 14, 12:25 PM
Unknown Object (File)
Thu, Apr 11, 8:29 AM
Unknown Object (File)
Thu, Apr 11, 3:13 AM
Unknown Object (File)
Sun, Apr 7, 3:16 PM
Unknown Object (File)
Sat, Apr 6, 2:21 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