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

	F15478041: D15776.id38008.diff
	Mon, Apr 7, 8:00 PM

	F15476748: D15776.id.diff
	Mon, Apr 7, 9:05 AM

	F15475016: D15776.diff
	Sun, Apr 6, 2:47 PM

	F15383834: D15776.id.diff
	Fri, Mar 14, 6:06 PM

	Unknown Object (File)
	Mar 3 2025, 2:00 PM

	Unknown Object (File)
	Feb 16 2025, 6:43 AM

	Unknown Object (File)
	Feb 12 2025, 4:50 PM

	Unknown Object (File)
	Feb 12 2025, 3:13 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

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...