Paths

Table of Contentst

Differential D15776

Fix `./bin/aphlict status`
ClosedPublic
Actions

Authored by joshuaspence on Apr 20 2016, 11:39 PM.

Tags

None

Referenced Files

	F14844945: D15776.id38009.diff
	Sun, Feb 2, 7:46 PM

	F14844199: D15776.diff
	Sun, Feb 2, 1:54 PM

	Unknown Object (File)
	Fri, Jan 24, 1:28 PM

	Unknown Object (File)
	Fri, Jan 24, 1:28 PM

	Unknown Object (File)
	Fri, Jan 24, 1:27 PM

	Unknown Object (File)
	Fri, Jan 24, 1:27 PM

	Unknown Object (File)
	Fri, Jan 24, 12:13 PM

	Unknown Object (File)
	Thu, Jan 23, 1:15 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T10844: `./bin/aphlict status` doesn't seem to be working

Commits

rP93e341fbdad3: Fix `./bin/aphlict status`

Summary

Fixes T10844. After recent changes to Aphlict (T6915 and T10697), ./bin/status needs to be aware of the configuration file. As such, it is now necessary to run ./bin/aphlict status --config /path/to/config.json rather than ./bin/aphlict status.

Test Plan

Ran ./bin/aphlict start ... and ./bin/aphlict status and saw "Aphlict ($PID) is running".

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 38008.Apr 20 2016, 11:39 PM

joshuaspence retitled this revision from to Fix `./bin/aphlict status`.

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

joshuaspence added a task: T10844: `./bin/aphlict status` doesn't seem to be working.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 20 2016, 11:39 PM

Herald added a subscriber: Korvin. · View Herald Transcript

epriestley accepted this revision.Apr 20 2016, 11:39 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 20 2016, 11:39 PM

Closed by commit rP93e341fbdad3: Fix `./bin/aphlict status` (authored by joshuaspence, committed by joshuaspence). · Explain WhyApr 20 2016, 11:45 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

aphlict/

management/

PhabricatorAphlictManagementStatusWorkflow.php

3 lines

Diff 38009

src/applications/aphlict/management/PhabricatorAphlictManagementStatusWorkflow.php

Loading...