1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-23 14:00:56 +01:00

Remove the "arc export" field from Differential revisions by default

Summary: This is kind of confusing (you need to specify an export format) and
not very useful now that "arc patch" has gotten pretty good. I'm leaving the
field itself in case installs want to add it back or otherwise depend on it.

Test Plan: Looked at a revision, wasn't told to export it.

Reviewers: nh, btrahan, jungejason

Reviewed By: nh

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1507
This commit is contained in:
epriestley 2012-01-27 13:02:11 -08:00
parent f009d5cdcb
commit bbf69309c0
2 changed files with 1 additions and 3 deletions

View file

@ -1,7 +1,7 @@
<?php <?php
/* /*
* Copyright 2011 Facebook, Inc. * Copyright 2012 Facebook, Inc.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -38,7 +38,6 @@ final class DifferentialDefaultFieldSelector
new DifferentialPathFieldSpecification(), new DifferentialPathFieldSpecification(),
new DifferentialArcanistProjectFieldSpecification(), new DifferentialArcanistProjectFieldSpecification(),
new DifferentialApplyPatchFieldSpecification(), new DifferentialApplyPatchFieldSpecification(),
new DifferentialExportPatchFieldSpecification(),
new DifferentialRevisionIDFieldSpecification(), new DifferentialRevisionIDFieldSpecification(),
new DifferentialGitSVNIDFieldSpecification(), new DifferentialGitSVNIDFieldSpecification(),
); );

View file

@ -13,7 +13,6 @@ phutil_require_module('phabricator', 'applications/differential/field/specificat
phutil_require_module('phabricator', 'applications/differential/field/specification/ccs'); phutil_require_module('phabricator', 'applications/differential/field/specification/ccs');
phutil_require_module('phabricator', 'applications/differential/field/specification/commits'); phutil_require_module('phabricator', 'applications/differential/field/specification/commits');
phutil_require_module('phabricator', 'applications/differential/field/specification/dependencies'); phutil_require_module('phabricator', 'applications/differential/field/specification/dependencies');
phutil_require_module('phabricator', 'applications/differential/field/specification/exportpatch');
phutil_require_module('phabricator', 'applications/differential/field/specification/gitsvnid'); phutil_require_module('phabricator', 'applications/differential/field/specification/gitsvnid');
phutil_require_module('phabricator', 'applications/differential/field/specification/host'); phutil_require_module('phabricator', 'applications/differential/field/specification/host');
phutil_require_module('phabricator', 'applications/differential/field/specification/lint'); phutil_require_module('phabricator', 'applications/differential/field/specification/lint');