fix: update dependency rollup to ^0.53.0 #438

renovate · 2017-12-22T15:26:44Z

This Pull Request updates dependency rollup from ^0.52.0 to ^0.53.0

Commits

rollup/rollup

0.53.0

3d80c06 Fix memory leak related to caching
9d43508 Do not redefine the default warning handler on each call.
5eec584 Fix code style
1b9913a 0.53.0
5feebb8 0.52.4
08b9b34 Update changelog
7084e30 Merge branch 'guybedford-dynamic-import-hook' into release-0.53.0
36ade1a * Minor cleanup to not display a warning due to the legacy input->entry fix
afd30df Errors out when using an unknown option
16d0a85 Removes batchWarnings from Rollup Node API
b354766 Removes filtering truthy values, uses spread instead.
d86afd7 Adds tests for deprecations.
d7b5d56 Fixes failing tests
5afda86 Uses mergeOptions to validate the config accross Node API, CLI and watch.
996fea6 Uses mergedOption in rollup.rollup too. (#1)
5f7bc9c Moves deprecateOptions, batchWarnings to src/utils
0c7f546 Removes destructuring as buble plugin includes source for rollup.js
ce5c5dc Passes arguments to onwarn correctly, fixes #1762

This PR has been generated by Renovate Bot.

TravisBuddy · 2017-12-22T15:30:54Z

Travis tests have failed

Hey Renovate Bot,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

Node.js: node

yarn test

yarn run v1.3.2
$ yarn build && yarn test:specs && yarn integration:samples && yarn integration:specs && yarn integration:consumer
$ rimraf dist && yarn schema
$ json2ts --input src/ng-package.schema.json --output src/ng-package.schema.ts
$ tsc -p src/tsconfig.packagr.json
$ node ./scripts/postbuild.js
$ cross-env TS_NODE_PROJECT=src/tsconfig.specs.json mocha --require ts-node/register src/**/*.spec.ts


  rollup
    
      ✓ should return 'false' paths starting with '.'
      ✓ should return 'false' for paths starting with '/'
      ✓ should return 'false' for absolute paths
      ✓ should return 'false' for embedded modules
      ✓ should return 'false' for 'commonjsHelpers'
      ✓ should return 'true' for external modules like '@angular/core'
      ✓ should return 'true' for modules with '.' like 'ui.core'
    
      ✓ should map 'rxjs/add/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/TeardownLogic' to 'Rx'
      ✓ should map 'rxjs/add/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/observable' to 'Rx.Observable'
      ✓ should map 'rxjs/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/operators' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operators/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/symbol' to 'Rx.Symbol'
      ✓ should map 'rxjs/symbol/iterator' to 'Rx.Symbol'
      ✓ should map 'rxjs/scheduler' to 'Rx.Scheduler'
      ✓ should map 'rxjs/scheduler/queue' to 'Rx.Scheduler'
      ✓ should map '@angular/core' to 'ng.core'
      ✓ should map '@angular/common/http' to 'ng.common.http'
      ✓ should map '@angular/platform-browser' to 'ng.platformBrowser'
      ✓ should map '@angular/platform-browser/animations' to 'ng.platformBrowser.animations'
      ✓ should map '@angular/platform-browser-dynamic' to 'ng.platformBrowserDynamic'
      ✓ should map 'foo' to 'FooModule' when 'umdModuleIds' is provided
      ✓ should map 'foo' to '' when no 'umdModuleIds' is provided

  ts-transformers
    isComponentDecorator
      ✓ should return true for '@Component()'
      ✓ should return false for other decorator
      ✓ should return false for non 'SyntaxKind.Decorator' nodes
    isTemplateUrl
      ✓ it should return 'true' for 'templateUrl:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes
    isStyleUrls
      ✓ it should return 'true' for 'stylesUrls:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes


  35 passing (20ms)

$ integration/samples.sh
Building sample ./integration/samples/core/ng-package.json...
Building Angular Package
Building from sources for entry point '@sample/core'
Cleaning build directory
Extracting templateUrl and styleUrls
Processing assets
Inlining templateUrl and styleUrls
Compiling with ngc
Bundling to FESM15

BUILD ERROR
Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
Error: Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
    at generate (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19112:44)
    at Object.write (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19161:13)
    at Object.<anonymous> (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:86:22)
    at Generator.next (<anonymous>)
    at fulfilled (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:4:58)
    at <anonymous>

Build failed. The working directory was not pruned. Files are stored at {ngPackage.workingDirectory}.
error Command failed with exit code 111.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Node.js: 8

yarn test

yarn run v1.3.2
$ yarn build && yarn test:specs && yarn integration:samples && yarn integration:specs && yarn integration:consumer
$ rimraf dist && yarn schema
$ json2ts --input src/ng-package.schema.json --output src/ng-package.schema.ts
$ tsc -p src/tsconfig.packagr.json
$ node ./scripts/postbuild.js
$ cross-env TS_NODE_PROJECT=src/tsconfig.specs.json mocha --require ts-node/register src/**/*.spec.ts


  rollup
    
      ✓ should return 'false' paths starting with '.'
      ✓ should return 'false' for paths starting with '/'
      ✓ should return 'false' for absolute paths
      ✓ should return 'false' for embedded modules
      ✓ should return 'false' for 'commonjsHelpers'
      ✓ should return 'true' for external modules like '@angular/core'
      ✓ should return 'true' for modules with '.' like 'ui.core'
    
      ✓ should map 'rxjs/add/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/TeardownLogic' to 'Rx'
      ✓ should map 'rxjs/add/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/observable' to 'Rx.Observable'
      ✓ should map 'rxjs/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/operators' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operators/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/symbol' to 'Rx.Symbol'
      ✓ should map 'rxjs/symbol/iterator' to 'Rx.Symbol'
      ✓ should map 'rxjs/scheduler' to 'Rx.Scheduler'
      ✓ should map 'rxjs/scheduler/queue' to 'Rx.Scheduler'
      ✓ should map '@angular/core' to 'ng.core'
      ✓ should map '@angular/common/http' to 'ng.common.http'
      ✓ should map '@angular/platform-browser' to 'ng.platformBrowser'
      ✓ should map '@angular/platform-browser/animations' to 'ng.platformBrowser.animations'
      ✓ should map '@angular/platform-browser-dynamic' to 'ng.platformBrowserDynamic'
      ✓ should map 'foo' to 'FooModule' when 'umdModuleIds' is provided
      ✓ should map 'foo' to '' when no 'umdModuleIds' is provided

  ts-transformers
    isComponentDecorator
      ✓ should return true for '@Component()'
      ✓ should return false for other decorator
      ✓ should return false for non 'SyntaxKind.Decorator' nodes
    isTemplateUrl
      ✓ it should return 'true' for 'templateUrl:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes
    isStyleUrls
      ✓ it should return 'true' for 'stylesUrls:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes


  35 passing (42ms)

$ integration/samples.sh
Building sample ./integration/samples/core/ng-package.json...
Building Angular Package
Building from sources for entry point '@sample/core'
Cleaning build directory
Extracting templateUrl and styleUrls
Processing assets
Inlining templateUrl and styleUrls
Compiling with ngc
Bundling to FESM15

BUILD ERROR
Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
Error: Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
    at generate (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19112:44)
    at Object.write (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19161:13)
    at Object.<anonymous> (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:86:22)
    at Generator.next (<anonymous>)
    at fulfilled (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:4:58)
    at <anonymous>

Build failed. The working directory was not pruned. Files are stored at {ngPackage.workingDirectory}.
error Command failed with exit code 111.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Node.js: 7

yarn test

yarn run v1.3.2
warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || >=8.0.0"
$ yarn build && yarn test:specs && yarn integration:samples && yarn integration:specs && yarn integration:consumer
warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || >=8.0.0"
$ rimraf dist && yarn schema
warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || >=8.0.0"
$ json2ts --input src/ng-package.schema.json --output src/ng-package.schema.ts
$ tsc -p src/tsconfig.packagr.json
$ node ./scripts/postbuild.js
warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || >=8.0.0"
$ cross-env TS_NODE_PROJECT=src/tsconfig.specs.json mocha --require ts-node/register src/**/*.spec.ts


  rollup
    
      ✓ should return 'false' paths starting with '.'
      ✓ should return 'false' for paths starting with '/'
      ✓ should return 'false' for absolute paths
      ✓ should return 'false' for embedded modules
      ✓ should return 'false' for 'commonjsHelpers'
      ✓ should return 'true' for external modules like '@angular/core'
      ✓ should return 'true' for modules with '.' like 'ui.core'
    
      ✓ should map 'rxjs/add/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/TeardownLogic' to 'Rx'
      ✓ should map 'rxjs/add/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/observable' to 'Rx.Observable'
      ✓ should map 'rxjs/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/operators' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operators/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/symbol' to 'Rx.Symbol'
      ✓ should map 'rxjs/symbol/iterator' to 'Rx.Symbol'
      ✓ should map 'rxjs/scheduler' to 'Rx.Scheduler'
      ✓ should map 'rxjs/scheduler/queue' to 'Rx.Scheduler'
      ✓ should map '@angular/core' to 'ng.core'
      ✓ should map '@angular/common/http' to 'ng.common.http'
      ✓ should map '@angular/platform-browser' to 'ng.platformBrowser'
      ✓ should map '@angular/platform-browser/animations' to 'ng.platformBrowser.animations'
      ✓ should map '@angular/platform-browser-dynamic' to 'ng.platformBrowserDynamic'
      ✓ should map 'foo' to 'FooModule' when 'umdModuleIds' is provided
      ✓ should map 'foo' to '' when no 'umdModuleIds' is provided

  ts-transformers
    isComponentDecorator
      ✓ should return true for '@Component()'
      ✓ should return false for other decorator
      ✓ should return false for non 'SyntaxKind.Decorator' nodes
    isTemplateUrl
      ✓ it should return 'true' for 'templateUrl:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes
    isStyleUrls
      ✓ it should return 'true' for 'stylesUrls:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes


  35 passing (25ms)

warning You are using Node "7.10.1" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || >=8.0.0"
$ integration/samples.sh
Building sample ./integration/samples/core/ng-package.json...
Building Angular Package
Building from sources for entry point '@sample/core'
Cleaning build directory
Extracting templateUrl and styleUrls
Processing assets
Inlining templateUrl and styleUrls
Compiling with ngc
Bundling to FESM15

BUILD ERROR
Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
Error: Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
    at generate (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19112:44)
    at Object.write (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19161:13)
    at Object.<anonymous> (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:86:22)
    at Generator.next (<anonymous>)
    at fulfilled (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:4:58)

Build failed. The working directory was not pruned. Files are stored at {ngPackage.workingDirectory}.
error Command failed with exit code 111.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Node.js: 6

yarn test

yarn run v1.3.2
$ yarn build && yarn test:specs && yarn integration:samples && yarn integration:specs && yarn integration:consumer
$ rimraf dist && yarn schema
$ json2ts --input src/ng-package.schema.json --output src/ng-package.schema.ts
$ tsc -p src/tsconfig.packagr.json
$ node ./scripts/postbuild.js
$ cross-env TS_NODE_PROJECT=src/tsconfig.specs.json mocha --require ts-node/register src/**/*.spec.ts


  rollup
    
      ✓ should return 'false' paths starting with '.'
      ✓ should return 'false' for paths starting with '/'
      ✓ should return 'false' for absolute paths
      ✓ should return 'false' for embedded modules
      ✓ should return 'false' for 'commonjsHelpers'
      ✓ should return 'true' for external modules like '@angular/core'
      ✓ should return 'true' for modules with '.' like 'ui.core'
    
      ✓ should map 'rxjs/add/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/TeardownLogic' to 'Rx'
      ✓ should map 'rxjs/add/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/observable' to 'Rx.Observable'
      ✓ should map 'rxjs/observable/bindCallback' to 'Rx.Observable'
      ✓ should map 'rxjs/operators' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operator/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/operators/audit' to 'Rx.Observable.prototype'
      ✓ should map 'rxjs/symbol' to 'Rx.Symbol'
      ✓ should map 'rxjs/symbol/iterator' to 'Rx.Symbol'
      ✓ should map 'rxjs/scheduler' to 'Rx.Scheduler'
      ✓ should map 'rxjs/scheduler/queue' to 'Rx.Scheduler'
      ✓ should map '@angular/core' to 'ng.core'
      ✓ should map '@angular/common/http' to 'ng.common.http'
      ✓ should map '@angular/platform-browser' to 'ng.platformBrowser'
      ✓ should map '@angular/platform-browser/animations' to 'ng.platformBrowser.animations'
      ✓ should map '@angular/platform-browser-dynamic' to 'ng.platformBrowserDynamic'
      ✓ should map 'foo' to 'FooModule' when 'umdModuleIds' is provided
      ✓ should map 'foo' to '' when no 'umdModuleIds' is provided

  ts-transformers
    isComponentDecorator
      ✓ should return true for '@Component()'
      ✓ should return false for other decorator
      ✓ should return false for non 'SyntaxKind.Decorator' nodes
    isTemplateUrl
      ✓ it should return 'true' for 'templateUrl:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes
    isStyleUrls
      ✓ it should return 'true' for 'stylesUrls:'
      ✓ it should return 'false' for other property assignment
      ✓ it should return 'false' for other nodes


  35 passing (28ms)

$ integration/samples.sh
Building sample ./integration/samples/core/ng-package.json...
Building Angular Package
Building from sources for entry point '@sample/core'
Cleaning build directory
Extracting templateUrl and styleUrls
Processing assets
Inlining templateUrl and styleUrls
Compiling with ngc
Bundling to FESM15

BUILD ERROR
Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
Error: Unknown option found: moduleId. Allowed keys: input, legacy, treeshake, acorn, context, moduleContext, plugins, onwarn, watch, cache, preferConst, experimentalDynamicImport, entry, external, extend, amd, banner, footer, intro, format, outro, sourcemap, sourcemapFile, name, globals, interop, legacy, freeze, indent, strict, noConflict, paths, exports, file, pureExternalModules
    at generate (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19112:44)
    at Object.write (/home/travis/build/dherges/ng-packagr/node_modules/rollup/dist/rollup.js:19161:13)
    at Object.<anonymous> (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:86:22)
    at next (native)
    at fulfilled (/home/travis/build/dherges/ng-packagr/dist/lib/steps/rollup.js:4:58)

Build failed. The working directory was not pruned. Files are stored at {ngPackage.workingDirectory}.
error Command failed with exit code 111.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

github-actions · 2020-06-19T16:08:52Z

This PR has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

_{This action has been performed automatically by a bot.}

dherges mentioned this pull request Dec 25, 2017

fix: remove moduleId from rollup bundle options #444

Merged

2 tasks

fix: update rollup to version ^0.53.0

42cb8b8

dherges force-pushed the renovate/rollup-0.x branch from 63dda9d to 42cb8b8 Compare December 25, 2017 14:31

dherges merged commit 8918809 into master Dec 25, 2017

dherges deleted the renovate/rollup-0.x branch December 25, 2017 14:38

github-actions bot locked and limited conversation to collaborators Jun 19, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: update dependency rollup to ^0.53.0 #438

fix: update dependency rollup to ^0.53.0 #438

renovate bot commented Dec 22, 2017

TravisBuddy commented Dec 22, 2017

github-actions bot commented Jun 19, 2020

fix: update dependency rollup to ^0.53.0 #438

fix: update dependency rollup to ^0.53.0 #438

Conversation

renovate bot commented Dec 22, 2017

Commits

0.53.0

TravisBuddy commented Dec 22, 2017

Travis tests have failed

Node.js: node

Node.js: 8

Node.js: 7

Node.js: 6

github-actions bot commented Jun 19, 2020