mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-19 12:00:55 +01:00
Make Releeph a normal prototype application
Summary: Fixes T6177. Now that we've reframed "Beta" into "Prototype", there's no reason this needs to be in a separate super-hidden class of application anymore. Test Plan: Saw Releeph available as a normal Prototype application. Reviewers: btrahan, chad Reviewed By: chad Subscribers: epriestley Maniphest Tasks: T6177 Differential Revision: https://secure.phabricator.com/D10550
This commit is contained in:
parent
93e1978e05
commit
cd4e3c6399
3 changed files with 2 additions and 21 deletions
|
@ -18,11 +18,8 @@ final class PhabricatorReleephApplication extends PhabricatorApplication {
|
|||
return 'releeph';
|
||||
}
|
||||
|
||||
public function isInstalled() {
|
||||
if (PhabricatorEnv::getEnvConfig('releeph.installed')) {
|
||||
return parent::isInstalled();
|
||||
}
|
||||
return false;
|
||||
public function isPrototype() {
|
||||
return true;
|
||||
}
|
||||
|
||||
public function getRoutes() {
|
||||
|
|
|
@ -42,19 +42,6 @@ final class PhabricatorReleephApplicationConfigOptions
|
|||
$custom_field_type = 'custom:PhabricatorCustomFieldConfigOptionType';
|
||||
|
||||
return array(
|
||||
$this->newOption('releeph.installed', 'bool', false)
|
||||
->setSummary(pht('Enable the Releeph application.'))
|
||||
->setDescription(
|
||||
pht(
|
||||
'Releeph, a tool for managing release branches, will eventually '.
|
||||
'fit in to the Phabricator suite as a general purpose tool. '.
|
||||
'However Releeph is currently unstable in multiple ways that may '.
|
||||
'not migrate properly for you: the code is still in alpha stage '.
|
||||
'of design, the storage format is likely to change in unexpected '.
|
||||
'ways, and the workflows presented are very specific to a core '.
|
||||
'set of alpha testers at Facebook. For the time being you are '.
|
||||
'strongly discouraged from relying on Releeph being at all '.
|
||||
'stable.')),
|
||||
$this->newOption('releeph.fields', $custom_field_type, $default)
|
||||
->setCustomData('ReleephFieldSpecification'),
|
||||
$this->newOption(
|
||||
|
|
|
@ -116,9 +116,6 @@ abstract class PhabricatorTestCase extends ArcanistPhutilTestCase {
|
|||
'notification.enabled',
|
||||
false);
|
||||
|
||||
// TODO: Remove this when we remove "releeph.installed".
|
||||
$this->env->overrideEnvConfig('releeph.installed', true);
|
||||
|
||||
$this->env->overrideEnvConfig(
|
||||
'phabricator.base-uri',
|
||||
'http://phabricator.example.com');
|
||||
|
|
Loading…
Reference in a new issue