Arcanist - fix shouldAmend code
ClosedPublic
Actions

Authored by btrahan on Apr 11 2014, 8:07 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, May 7, 6:33 AM

	Unknown Object (File)
	Fri, May 3, 5:51 AM

	Unknown Object (File)
	Mon, Apr 29, 4:28 PM

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

	Unknown Object (File)
	Mon, Apr 15, 3:38 AM

	Unknown Object (File)
	Mon, Apr 15, 3:38 AM

	Unknown Object (File)
	Mon, Apr 15, 3:20 AM

	Unknown Object (File)
	Mon, Apr 15, 3:02 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley
shadowhand

Commits

rARC43792895a679: Arcanist - fix shouldAmend code

Summary

make it a real member variable. See rARC77a9c1814063 and D8753#33914.

Test Plan

sending up this diff!

Diff Detail

Repository

rARC Arcanist

Branch

fixboooooo

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

btrahan updated this revision to Diff 20779.Apr 11 2014, 8:07 PM

btrahan retitled this revision from to Arcanist - fix shouldAmend code.

btrahan updated this object.

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

btrahan added reviewers: epriestley, shadowhand.

Herald added subscribers: Korvin, epriestley.Apr 11 2014, 8:07 PM

epriestley accepted this revision.Apr 11 2014, 8:07 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 11 2014, 8:07 PM

Closed by commit rARC43792895a679 (authored by @btrahan).

Makes more sense now. Thanks. :)

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistBaseWorkflow.php

12 lines

Diff 20779

View Options

src/workflow/ArcanistBaseWorkflow.php

Arcanist - fix shouldAmend codeClosedPublicActions