Set celerity.minify true in production environments.

Summary: I haven't been able to understand why this isn't set by default in production environments (since it is recommended to do so anyway).

Test Plan: N/A

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D8743

Author: joshuaspence

conf/default.conf.php

@@ -980,7 +980,7 @@
 
   // Minify static resources by removing whitespace and comments. You should
   // enable this in production, but disable it in development.
-  'celerity.minify' => false,
+  'celerity.minify' => true,
 
   // You can respond to various application events by installing listeners,
   // which will receive callbacks when interesting things occur. Specify a list

conf/development.conf.php

@@ -4,5 +4,6 @@
 
   'phabricator.developer-mode'      => true,
   'darkconsole.enabled'             => true,
+  'celerity.minify'                 => false,
 
 ) + phabricator_read_config_file('default');

src/applications/config/option/PhabricatorDeveloperConfigOptions.php

@@ -97,7 +97,7 @@ public function getOptions() {
               "This option enables verbose error reporting (stack traces, ".
               "error callouts) and forces disk reads of static assets on ".
               "every reload.")),
-      $this->newOption('celerity.minify', 'bool', false)
+      $this->newOption('celerity.minify', 'bool', true)
         ->setBoolOptions(
           array(
             pht('Minify static resources.'),