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

Make compressors more usable from Java context #257

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Make compressors more usable from Java context #257

wants to merge 2 commits into from

Conversation

vigorouscoding
Copy link

This makes YUICompressor more flexible for usage in other software products.
It enables the user to pass JS/CSS Strings instead of files to the compressors - it also enables the user to grab the minified code without having to write it to a file.

It does not impact the functionality of the compressors at all, it merely adds layer to the compressors which can act as a point of extension or can be used as is to simply retrieve the compressed JS/CSS without reading/writing it from/to a file.

All tests pass since no extra functionality is introduced. I tried my best to adopt your coding style and indentation rules and hope you think this code benefits YUICompressor :)

vigorouscoding added 2 commits August 22, 2016 15:58
- introduce a constructor which takes a String argument so we do not need a file
- move actual CSS compression to method `getCompressedCss()` so we can obtain the minified CSS without needing to write it to a file
- introduced a new constructr which takes a String argument so we do not need a file
- move actuall JS compression to `getCompressedCode()` method so we can obtain the minified JS without needing to write to a file
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.

1 participant