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

	Unknown Object (File)
	Tue, May 27, 11:05 AM

	Unknown Object (File)
	Fri, May 9, 9:45 AM

	Unknown Object (File)
	Fri, May 9, 5:19 AM

	Unknown Object (File)
	Thu, May 8, 3:34 PM

	Unknown Object (File)
	Tue, May 6, 11:55 PM

	Unknown Object (File)
	Apr 29 2025, 5:12 AM

	Unknown Object (File)
	Apr 17 2025, 11:42 PM

	Unknown Object (File)
	Apr 10 2025, 12:07 AM

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

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 11879
Build 14918: Run Core Tests
Build 14917: arc lint + arc unit

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

Coverage (All)

Coverage (Touched)

src/

applications/

aphlict/

management/

PhabricatorAphlictManagementStatusWorkflow.php

3 lines

13%

Loading...

Diff 38008

src/applications/aphlict/management/PhabricatorAphlictManagementStatusWorkflow.php

Loading...