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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace uglify-es with uglify-js #644

Closed
SimenB opened this issue Mar 6, 2021 · 2 comments
Closed

Replace uglify-es with uglify-js #644

SimenB opened this issue Mar 6, 2021 · 2 comments

Comments

@SimenB
Copy link
Contributor

SimenB commented Mar 6, 2021

Do you want to request a feature or report a bug?

Bug? Dunno 馃槄

What is the current behavior?

"uglify-es": "^3.1.9"

uglify-es@npm:3.3.10 is deprecated: support for ECMAScript is superseded by `uglify-js` as of v3.13.0

https://www.npmjs.com/package/uglify-es

If the current behavior is a bug, please provide the steps to reproduce and a minimal repository on GitHub that we can yarn install and yarn test.

Just do yarn install metro

What is the expected behavior?

No deprecation warning when installing

Please provide your exact Metro configuration and mention your Metro, node, yarn/npm version and operating system.

N/A

@SakuOtonashi
Copy link

@robhogan
Copy link
Contributor

Effectively closed by #871 - we've now changed the default to terser instead.

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 a pull request may close this issue.

3 participants