Skip to content

Added a new mangleProperties option#910

Closed
ghost wants to merge 1 commit intomasterfrom
unknown repository
Closed

Added a new mangleProperties option#910
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 2, 2016

No description provided.

@neersighted
Copy link
Copy Markdown

This fixes #788 and is relevant to terinjokes/gulp-uglify#115.

@rvanvelzen
Copy link
Copy Markdown
Collaborator

This doesn't include the caching bit, which is quite important.

Also, the toplevel.figure_out_scope() shouldn't be needed.

@ffabreti
Copy link
Copy Markdown

ffabreti commented Jan 8, 2016

please merge...

@rvanvelzen
Copy link
Copy Markdown
Collaborator

Could you squash this into a single commit? I'd be happy to merge it then. :-)

@kdelmonte
Copy link
Copy Markdown

@jamarzka could you please squash so we can use this?

@kdelmonte
Copy link
Copy Markdown

Actually, looks like he already did. :)

@kdelmonte
Copy link
Copy Markdown

@rvanvelzen , any update on this?

@rvanvelzen
Copy link
Copy Markdown
Collaborator

Hmm, github doesn't notify me of updates. I'll merge this in a bit :)

@rvanvelzen
Copy link
Copy Markdown
Collaborator

Merged as 799509e, thanks!

@rvanvelzen rvanvelzen closed this Jan 26, 2016
@kdelmonte
Copy link
Copy Markdown

Thanks.

@kdelmonte
Copy link
Copy Markdown

@rvanvelzen , one more thing. When will this be pushed to npm?

@kdelmonte
Copy link
Copy Markdown

@rvanvelzen , please see above.

@rvanvelzen
Copy link
Copy Markdown
Collaborator

@kdelmonte @mishoo does releases, I don't. Won't be too long from now, probably.

@kdelmonte
Copy link
Copy Markdown

Thank you.

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.

4 participants