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

separate attribution comment & use "/*!" for it #80

Closed
wants to merge 1 commit into from
Closed

separate attribution comment & use "/*!" for it #80

wants to merge 1 commit into from

Conversation

cvrebert
Copy link
Contributor

Use "/*!" to ensure attribution comment is preserved by minifiers.

Use "/*!" to ensure attribution comment is preserved by minifiers
@dduponchel
Copy link
Collaborator

Thanks for the pull request ! I just merged #74 which changes a lot of things regarding the generated/minified file (now we use grunt + browserify + uglify). The issue remains (the license is lost) but your commit is not enough anymore.
I fixed the grunt configuration to ensure that your fix is effective and I will create a pull request including your commit. As soon as @Stuk reviews/accepts it, this pull request will be automatically resolved.

dduponchel added a commit to dduponchel/jszip that referenced this pull request Jan 15, 2014
This commit updates the grunt configuration to ensure that the license
comment (starting with /*! for the minifiers) makes its way to the final
dist files.
@cvrebert
Copy link
Contributor Author

Closing per #81.

@cvrebert cvrebert closed this Jan 15, 2014
@cvrebert cvrebert deleted the patch-1 branch January 15, 2014 23:49
dduponchel added a commit that referenced this pull request Feb 1, 2014
change the grunt config for the issue #80
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