-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Minify index.html on build --prod #1861
Comments
Any update on this? |
Is there chance to minify html? |
Please add an option to turn off HTML minification. Or, at least, whitespace collapsing. |
Does this minify all angular templates or just |
@e-oz is this change breaking something in your project? Can you open a new issue with a description of what's happening please? @ciesielskico just index.html minification. |
if it's only for index.html - no, nothing critical :)
…On Mon, 12 Jun 2017 at 12:40, Filipe Silva ***@***.***> wrote:
@e-oz <https://github.com/e-oz> is this change breaking something in your
project? Can you open a new issue with a description of what's happening
please?
@ciesielskico <https://github.com/ciesielskico> just index.html
minification.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1861 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAgIEO9IBj4v98jGm5l8ihw3W00oSpJ1ks5sDRWAgaJpZM4Ju05w>
.
|
Why this issue was closed? Is there any option to minify index.html? It would be great to have an option like "extractCss" to enabled/disabled minify index.html |
It was closed because of this commit 75311c2 |
@odahcam This feature was removed on Angular 6+ based on the comments noted on the commit you have linked. So |
Absolutely right, I read the comments too, I just wanted to give a direct and simple answer to the previous comment. Maybe this issue should be reopened. |
@odahcam I agree. It should be. @filipesilva thoughts? |
#11360 is the same issue, and it's still open, in case you guys want to move there. |
Any updates on this? |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
ng build --prod
It does not minify
index.html
The text was updated successfully, but these errors were encountered: