Skip to content

build(rollup): replace rollup plugin uglify with terser#179

Merged
remarkablemark merged 1 commit intomasterfrom
build/fix-npm-install
Nov 1, 2020
Merged

build(rollup): replace rollup plugin uglify with terser#179
remarkablemark merged 1 commit intomasterfrom
build/fix-npm-install

Conversation

@remarkablemark
Copy link
Copy Markdown
Owner

What is the motivation for this pull request?

Fix Travis CI build error when installing with npm 7:

$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found: rollup@2.32.1
npm ERR! node_modules/rollup
npm ERR!   dev rollup@"^2.12.1" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer rollup@">=0.66.0 <2" from rollup-plugin-uglify@6.0.4
npm ERR! node_modules/rollup-plugin-uglify
npm ERR!   dev rollup-plugin-uglify@"^6.0.4" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

To resolve peerDependencies error and support rollup v2, replace rollup-plugin-uglify with rollup-plugin-terser.

See TrySound/rollup-plugin-uglify#82

Fix error when installing with npm 7:

```sh
$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found: rollup@2.32.1
npm ERR! node_modules/rollup
npm ERR!   dev rollup@"^2.12.1" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer rollup@">=0.66.0 <2" from rollup-plugin-uglify@6.0.4
npm ERR! node_modules/rollup-plugin-uglify
npm ERR!   dev rollup-plugin-uglify@"^6.0.4" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
```
@remarkablemark remarkablemark added the dependencies Pull requests that update a dependency file label Nov 1, 2020
@remarkablemark remarkablemark merged commit ab41ff5 into master Nov 1, 2020
@remarkablemark remarkablemark deleted the build/fix-npm-install branch November 1, 2020 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant