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

Update rollup-plugin-uglify to the latest version 🚀 #238

Closed
wants to merge 1 commit into from

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented May 19, 2018

Version 4.0.0 of rollup-plugin-uglify was just published.

Dependency rollup-plugin-uglify
Current Version 3.0.0
Type devDependency

The version 4.0.0 is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.

It might be worth looking into these changes and trying to get this project onto the latest version of rollup-plugin-uglify.

If you have a solid test suite and good coverage, a passing build is a strong indicator that you can take advantage of these changes directly by merging the proposed change into your project. If the build fails or you don’t have such unconditional trust in your tests, this branch is a great starting point for you to work on the update.


Release Notes v4 and back to es5

uglify-es is not supported anymore so we returned back to old school uglify-js. If you still want to transpile es6+ try rollup-plugin-terser. Terser is actively developed fork of uglify-es with the new name.

In this version we changed the way to import plugin. Now it looks like this

import { uglify } from 'rollup-plugin-uglify';

export default {
...
plugins: [uglify()]
...
}

To see more prettier errors uglify shows also babel code frame

    > 1 | var = 1
        |    ^ Name expected

Error: Error transforming bundle with 'uglify' plugin: Name expected

Commits

The new version differs by 6 commits.

  • a847d2e 4.0.0
  • cd8f9f1 Change default export to named one
  • 6e46a12 Use uglify-js as uglify-es got deprecated in favour of terser (#42)
  • 3beec87 Merge pull request #40 from kzc/patch-1
  • 5c07826 have README point to uglify-es docs
  • 3dc7600 Fix tests

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 🌴

@julkue julkue self-assigned this Jun 21, 2018
Copy link
Owner

@julkue julkue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@julkue julkue closed this in e3251c8 Jun 21, 2018
@julkue julkue deleted the greenkeeper/rollup-plugin-uglify-4.0.0 branch June 21, 2018 18:53
@julkue
Copy link
Owner

julkue commented Jun 21, 2018

Even though asked why here.

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.

1 participant