Skip to content

Added a new mangleProperties option#910

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

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

Conversation

@ghost

@ghost ghost commented Jan 2, 2016

Copy link
Copy Markdown

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

ffabreti commented Jan 8, 2016

Copy link
Copy Markdown

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