Differential D20922

Add a "--dry-run" flag to "bin/repository rebuild-identities"
ClosedPublic
Actions

Authored by epriestley on Nov 19 2019, 8:34 PM.

Details

Reviewers: None
Maniphest Tasks: T13444: Provide a standalone script entry point for resolving a repository identity
Commits: rP374f8b10b3cf: Add a "--dry-run" flag to "bin/repository rebuild-identities"

Summary

Ref T13444. Allow the effects of performing an identity rebuild to be previewed without committing to any changes.

Test Plan

Ran "bin/repository rebuild-identities --all-identities" with and without "--dry-run".

Diff Detail

Repository

rP Phabricator

Branch

identity2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 23690
Build 32570: Run Core Tests
Build 32569: arc lint + arc unit

Event Timeline

epriestley created this revision.Nov 19 2019, 8:34 PM

Harbormaster completed remote builds in B23690: Diff 49861.Nov 19 2019, 8:36 PM

epriestley requested review of this revision.Nov 19 2019, 8:36 PM

This revision was not accepted when it landed; it landed in state Needs Review.Nov 19 2019, 8:38 PM

Closed by commit rP374f8b10b3cf: Add a "--dry-run" flag to "bin/repository rebuild-identities" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP374f8b10b3cf: Add a "--dry-run" flag to "bin/repository rebuild-identities".

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

identity/

DiffusionRepositoryIdentityEngine.php

20 lines

repository/

management/

PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php

89 lines

Diff 49861

View Options

src/applications/diffusion/identity/DiffusionRepositoryIdentityEngine.php