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

Upgrade terser to version 5 #12454

Merged
merged 1 commit into from
Oct 7, 2020
Merged

Conversation

Snuffleupagus
Copy link
Collaborator

The only significant change is that the minify command is now asynchronous, which we can handle easily by simply making the containing functions async.
Based on https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function#Browser_compatibility, using async/await in the gulpfile should no longer be an issue as far as I can tell.

The only significant change is that the `minify` command is now asynchronous, which we can handle easily by simply making the containing functions `async`.
Based on https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function#Browser_compatibility, using `async`/`await` in the gulpfile should no longer be an issue as far as I can tell.
@Snuffleupagus
Copy link
Collaborator Author

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Oct 7, 2020

From: Bot.io (Linux m4)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.67.70.0:8877/b245cef36e05ef1/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Oct 7, 2020

From: Bot.io (Linux m4)


Success

Full output at http://54.67.70.0:8877/b245cef36e05ef1/output.txt

Total script time: 3.53 mins

Published

@timvandermeij timvandermeij merged commit 99a2302 into mozilla:master Oct 7, 2020
@timvandermeij
Copy link
Contributor

Nice, another one done!

@Snuffleupagus Snuffleupagus deleted the terser-5 branch October 8, 2020 08:05
@Snuffleupagus
Copy link
Collaborator Author

Thanks for landing this, and the other one too!

I'm not going to attempt the yargs upgrade though, since compared to the others I cannot immediately see what changes are necessary and because this dependency isn't part of the build process (and thus slightly less important IMHO).

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.

3 participants