Skip to content
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

Add plugin: HtmlMinPlugin for minifying HTML. #254

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tasn
Copy link

@tasn tasn commented Mar 3, 2014

This plugin uses the htmlmin plugin to minify html.
It's very simple, but supports all of the htmlmin configurations.

There are no unit tests, as this is a wrapper for the htmlmin library.

@tasn
Copy link
Author

tasn commented Mar 4, 2014

Actually, wait, I'm fixing some things, will send something else soon.

@tasn tasn closed this Mar 4, 2014
@tasn
Copy link
Author

tasn commented Mar 4, 2014

OK, fixed anything, it's good now.

@tasn tasn reopened this Mar 4, 2014
This plugin uses the htmlmin plugin to minify html.
It's very simple, but supports all of the htmlmin configurations.
@tasn
Copy link
Author

tasn commented Aug 19, 2014

Ping.

@llonchj
Copy link
Contributor

llonchj commented May 23, 2015

Hello @tasn! This plugin looks amazing.
Can you please write some unit tests so we can merge this plugin?

@tasn
Copy link
Author

tasn commented May 25, 2015

Hey,

Thanks for replying, and glad to see someone is finally taking care of this. However, it has been a year, and I've already switched away from Hyde (lack of development/response), so I have no interest in updating this patch with tests. Feel free to still merge it and add the tests on your own if you find this behaviour desirable.

Tom.

@llonchj
Copy link
Contributor

llonchj commented May 25, 2015

Thanks @tasn. Will do some tests and include your plugin in the next release.

@llonchj llonchj added this to the Version 0.9.0 milestone May 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants