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

Authored by epriestley on Jun 20 2019, 9:41 PM.

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

Event Timeline

epriestley created this revision.Jun 20 2019, 9:41 PM

Harbormaster completed remote builds in B23013: Diff 49141.Jun 20 2019, 9:41 PM

epriestley requested review of this revision.Jun 20 2019, 9:41 PM

amckinley accepted this revision.Jun 20 2019, 10:10 PM

This revision is now accepted and ready to land.Jun 20 2019, 10:10 PM

Closed by commit rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land" (authored by epriestley). · Explain WhyJun 20 2019, 11:07 PM

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCd92fa96366c0: Fix two "msort()" vs "msortv()" issues in "arc land".

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistLandWorkflow.php

4 lines

Diff 49141

View Options

src/workflow/ArcanistLandWorkflow.php

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