Skip to content
This repository has been archived by the owner on Jul 27, 2018. It is now read-only.

feat(build): Upgrade build process to emit ESM modules and a UMD bundle #20

Merged
merged 1 commit into from
Sep 15, 2016

Conversation

brandonroberts
Copy link
Member

No description provided.

'rxjs/operator/distinctUntilChanged': 'Rx.Observable.prototype',
'rxjs/operator/do': 'Rx.Observable.prototype',
'rxjs/operator/filter': 'Rx.Observable.prototype',
'rxjs/operator/map': 'Rx.Observable.prototype'
Copy link
Member

Choose a reason for hiding this comment

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

You will need to map @ngrx/core here

'@ngrx/core': 'ngrx.core'

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

entry: './release/index.js',
dest: './release/bundles/router-store.umd.js',
format: 'umd',
moduleName: 'ngrx.router-store',
Copy link
Member

@MikeRyanDev MikeRyanDev Sep 15, 2016

Choose a reason for hiding this comment

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

The hyphen here won't work, you will need to camel-case this to ngrx.routerStore

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@@ -1,22 +1,23 @@

Copy link
Member

Choose a reason for hiding this comment

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

This legacy file was only needed for @ngrx/core to preserve deep imports. You should not need a legacy build step.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@MikeRyanDev MikeRyanDev merged commit 2bbf437 into master Sep 15, 2016
@MikeRyanDev MikeRyanDev deleted the chore-build branch September 15, 2016 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants