Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun`
bcd87e0e3f37
Actions

Description

Don't apply patches or mark patches applied with bin/storage upgrade --dryrun

Summary: Fixes T12682.

Test Plan: Ran bin/storage upgrade --dryrun repeatedly with un-applied patches, saw it not apply them and not mark them applied.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12682

Differential Revision: https://secure.phabricator.com/D17837

Details

Provenance

epriestley	Authored on May 6 2017, 2:55 AM
epriestley	Pushed on May 6 2017, 2:57 AM

Reviewer

chad

Differential Revision

D17837: Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun`

Parents

rP0a87881e9813: Fix file attach bug in Macro

Branches

Unknown

Tags

Unknown

Tasks

T12682: `storage --dry-run` actually looks rather wet when applying patches

Build Status

Buildable 16837
Build 22472: Run Core Tests

Event Timeline

epriestley committed rPbcd87e0e3f37: Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun` (authored by epriestley).May 6 2017, 2:57 AM

epriestley added a task: T12682: `storage --dry-run` actually looks rather wet when applying patches.

Harbormaster completed building B16837: rPbcd87e0e3f37: Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun`.May 6 2017, 2:59 AM

joshuaspence mentioned this in T12689: Mail is still received after resigning from a revision.May 8 2017, 10:37 PM

Changes (1)

Path

Size

src/

infrastructure/

storage/

management/

workflow/

PhabricatorStorageManagementWorkflow.php

rPbcd87e0e3f37

View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php