Page MenuHomePhabricator

Set `celerity.minify` true in production environments.
ClosedPublic

Authored by joshuaspence on Apr 10 2014, 2:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 8:21 PM
Unknown Object (File)
Fri, Dec 6, 8:15 AM
Unknown Object (File)
Fri, Dec 6, 6:49 AM
Unknown Object (File)
Tue, Nov 26, 12:30 AM
Unknown Object (File)
Tue, Nov 26, 12:18 AM
Unknown Object (File)
Oct 26 2024, 7:25 AM
Unknown Object (File)
Oct 24 2024, 3:52 PM
Unknown Object (File)
Oct 19 2024, 2:54 AM
Subscribers

Details

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

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Set `celerity.minify` true in production environments..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Nice catch!

This file is very rarely read now, so this isn't the best way to adjust the defaults (it will apply to only some installs). Instead:

  • Adjust the global default in src/applications/config/option/PhabricatorDeveloperConfigOptions.php.
  • Adjust the default from conf/default.conf.php, or remove it entirely (if removed, it will fall back to the global default).
  • Override it in conf/development.conf.php.

I think this never got set to true by default because it used to rely on building complicated externals (notably, jsxmin). However, we now have reasonable builtin/fallback minifiers and setting it to true is sensible.

This revision now requires changes to proceed.Apr 10 2014, 2:19 PM
joshuaspence edited edge metadata.
  • Adjust the global default in src/applications/config/option/PhabricatorDeveloperConfigOptions.php.
  • Adjust the default from conf/default.conf.php.
  • Override it in conf/development.conf.php.
epriestley edited edge metadata.

(Update libphutil/ to clear lint, first bullet point in the message.)

This revision is now accepted and ready to land.Apr 10 2014, 9:38 PM
epriestley updated this revision to Diff 20757.

Closed by commit rPe7419d85f700 (authored by @joshuaspence, committed by @epriestley).