Arcanist - allow amending revisions by other authors in the working copy
ClosedPublic
Actions

Authored by btrahan on Apr 7 2014, 7:04 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 4, 6:32 AM

	Unknown Object (File)
	Fri, May 3, 7:14 AM

	Unknown Object (File)
	Sun, Apr 28, 11:41 AM

	Unknown Object (File)
	Sun, Apr 28, 11:40 AM

	Unknown Object (File)
	Sun, Apr 28, 11:40 AM

	Unknown Object (File)
	Wed, Apr 24, 10:43 PM

	Unknown Object (File)
	Thu, Apr 11, 8:48 AM

	Unknown Object (File)
	Thu, Apr 11, 8:48 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T4670: When we identify working copy revisions, we should annotate unowned revisions, not exclude them

Commits

rARC2714395d986c: Arcanist - allow amending revisions by other authors in the working copy

Summary

Fixes T4670. Give the user an option to abort but otherwise proceed.

Test Plan

arc patch D8685 (by epriestley); arc amend

Diff Detail

Repository

rARC Arcanist

Branch

T4670

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

btrahan updated this revision to Diff 20668.Apr 7 2014, 7:04 PM

btrahan added a task: T4670: When we identify working copy revisions, we should annotate unowned revisions, not exclude them.

btrahan retitled this revision from to Arcanist - allow amending revisions by other authors in the working copy.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley.Apr 7 2014, 7:04 PM

epriestley accepted this revision.Apr 8 2014, 2:56 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 8 2014, 2:56 AM

Closed by commit rARC2714395d986c (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistAmendWorkflow.php

19 lines

Diff 20668

View Options

src/workflow/ArcanistAmendWorkflow.php

Arcanist - allow amending revisions by other authors in the working copyClosedPublicActions