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] File merge function #282

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

Conversation

kang0921ok
Copy link

@kang0921ok kang0921ok commented Jun 3, 2017

I added a file merge function.

I used BufferWriter.

It is sovled #255

AS-IS
a.js b.js c.js -o min.js
--> c.js == min.js

T0-BE
a.js b.js c.js -o min.js
--> a.js + b.js + c.js == min.js

Thanks

@tml
Copy link
Contributor

tml commented Aug 18, 2017

Thanks for the contribution!

There are a handful of whitespace-only changes in this commit (3 changes to indentation level and a missing file-terminating newline); could you update the PR to remove those? If the whitespace needs to change for logical/consistency reasons, I'd prefer keeping those in distinct whitespace-only commits.

[ADD] File merge function

[ADD] File merge function

Update YUICompressor.java
@kang0921ok
Copy link
Author

Okay, I remove whitespace.
Check Plz man~

Thank you for confirming my PR.

@kang0921ok
Copy link
Author

@tml But This is not concerned That Same variable name. is It ok?

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.

2 participants