-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
Bump rollup-plugin-terser from 5.1.3 to 5.2.0 #19186
Bump rollup-plugin-terser from 5.1.3 to 5.2.0 #19186
Conversation
@material-ui/core: parsed: +0.44% , gzip: +0.09% Details of bundle changes.Comparing: ca8c102...39d7c98
|
Bumps [rollup-plugin-terser](https://github.com/TrySound/rollup-plugin-terser) from 5.1.3 to 5.2.0. - [Release notes](https://github.com/TrySound/rollup-plugin-terser/releases) - [Commits](TrySound/rollup-plugin-terser@v5.1.3...v5.2.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
a7e35c9
to
39d7c98
Compare
resolved "https://registry.yarnpkg.com/terser/-/terser-4.1.2.tgz#b2656c8a506f7ce805a3f300a2ff48db022fa391" | ||
integrity sha512-jvNoEQSPXJdssFwqPSgWjsOrb+ELoE+ILpHPKXC83tIxOlh2U75F1KuB2luLD/3a6/7K3Vw5pDn+hvu0C4AzSw== | ||
terser@3.16.1, terser@^4.1.2, terser@^4.6.2: | ||
version "4.6.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bisecting the upgrade to terser
to understand what's happening...
I suspect this is necessary to prevent bugs in certain edge cases but the increase is larger than I would expect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Biggest bump from 4.2.1
to 4.3.0
. Still upgrading to be safe. Next time this happens I'll take a look what exactly happened and if we can help terser by specifying options.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dependabot squash and merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this release I enabled toplevel by default for cjs format. Can it be related?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bump happens when only updating terser
as well which is why our docs bundle size also increased. As I said most of the increased happens when bumping terser
from 4.2.1 to 4.3.0.
Bumps rollup-plugin-terser from 5.1.3 to 5.2.0.
Release notes
Sourced from rollup-plugin-terser's releases.
Commits
b683311
v5.2.05c43828
Upgrade dependencies4b50a48
Add note about default options5503c23
Enable toplevel by default for cjs format and allow to override optionsf23a346
Fix nameCache (#62)9243334
Upgrade rollup and fix testDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot badge me
will comment on this PR with code to add a "Dependabot enabled" badge to your readmeAdditionally, you can set the following in the
.dependabot/config.yml
file in this repo: