Skip to content

Conversation

@williamareynolds
Copy link
Collaborator

No description provided.

format: 'umd',
globals: {
rxjs: 'rxjs',
'rxjs/operators': 'rxjs/operators'
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Kind of wondering if this is a valid way to provide the global symbol for rxjs operators, or if it should be shortened to operators or something.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Results in this in the UMD output:

(global = global || self, factory(global.monads = {}, global.rxjs, global['rxjs/operators']));

Copy link
Owner

Choose a reason for hiding this comment

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

I don't know the answer to this one. I think the way you have it is good.

Copy link
Owner

@patrickmichalina patrickmichalina left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@williamareynolds williamareynolds merged commit 07ff80b into master Mar 3, 2020
@williamareynolds williamareynolds deleted the maybe-to-observable branch March 3, 2020 23:36
@patrickmichalina
Copy link
Owner

🎉 This PR is included in version 3.16.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants