Page MenuHomePhabricator

Run setup checks from the CLI
Needs ReviewPublic

Authored by joshuaspence on Jun 18 2015, 12:04 PM.
Tags
None
Referenced Files
F14385438: D13336.diff
Sat, Dec 21, 11:40 AM
Unknown Object (File)
Tue, Dec 17, 9:59 AM
Unknown Object (File)
Fri, Dec 13, 6:38 PM
Unknown Object (File)
Fri, Dec 13, 9:15 AM
Unknown Object (File)
Fri, Dec 13, 9:15 AM
Unknown Object (File)
Fri, Dec 13, 8:55 AM
Unknown Object (File)
Fri, Dec 13, 7:34 AM
Unknown Object (File)
Thu, Dec 5, 10:44 AM
Subscribers

Details

Summary

Fixes T2383. Allows setup checks to be run from the CLI with ./bin/config check. The general idea is to split the PhabricatorSetupCheck::executeChecks method into three smaller components:

  • PhabricatorSetupCheck::executeCommandLineChecks is used for running setup checks which are specific to the command line. We don't really have any of these, but this diff makes it possible for us to add these types of checks.
  • PhabricatorSetupCheck::executeWebChecks is used for running setup checks which only make sense in a web context.
  • PhabricatorSetupCheck::executeCommonChecks is used to execute common checks shared between the command-line and web.
Test Plan

This probably needs some more work, but seeking some initial feedback.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6955
Build 6977: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Run setup checks from the CLI.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence edited edge metadata.

Show table borders