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

tests: for --optimization-mangle-exports & --optimization-used-exports #1729

Closed
wants to merge 1 commit into from

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented Aug 3, 2020

What kind of change does this PR introduce?
tests, fix

Did you add tests for your changes?
yes
If relevant, did you update the documentation?
NA
Summary
with the new release of webpack 5 new flags --optimization-mangle-exports & --optimization-used-exports are supported now. handling test cases for the same.

Does this PR introduce a breaking change?
No

Other information

@anshumanv
Copy link
Member

@evilebottnawi do we need test with mangleExports in config as raised in #1677? If not we can close the config PR.

@alexander-akait
Copy link
Member

@anshumanv more tests are better

@anshumanv
Copy link
Member

Okay, will keep it open, edited your comment @snitin315

@alexander-akait
Copy link
Member

Please regenerate lock file

@anshumanv
Copy link
Member

Cant regenerate because we're on webpack latest, for next it's only generated while installing in CI. Hence when webpack releases with updated terser rerunning in all PRs will fix. /cc @snitin315

@alexander-akait
Copy link
Member

@anshumanv Can you try to do it?

@anshumanv
Copy link
Member

Closing in favour of #1732

@anshumanv anshumanv closed this Aug 4, 2020
@anshumanv anshumanv deleted the fix/webpack-5-tests branch August 4, 2020 16:11
@anshumanv
Copy link
Member

@snitin315 quick question, what's the approach you followed to trace this back? Just curious

@snitin315
Copy link
Member Author

Theese flags or the terser warning?

@anshumanv
Copy link
Member

terser warning

@snitin315
Copy link
Member Author

webpack --node-args --trace-depriciation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants