-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minification should be done with PHP #1
Comments
What wrong with Minify?
I created a folder to test minifiers (59c7019). For the time being, it runs UglifyJS and JSMin as found in Minify. It's easy to add minifiers to test. I undertand the problem with PAAS only providing a single platform and I'd love to have a PHP-based minifier. However, as it stands now, a node web service close enough to the FullStack install would outperform current "state-of-the-art" PHP minifiers and divide minified file size by 2. Keep in mind that even if using UglifyJS wasn't competitive in term of performance, since we will cache the output, minification effectiveness should still be the deciding factor. |
Just a reminder note and a discussion entry point on the minification task. What is the problem with Minify ? Minification should be done with PHP because many PAAS are "platform exclusive". The NodeJS alternative (2ef87ad) is temporarily acceptable but couldn't be part of the final release.
The text was updated successfully, but these errors were encountered: