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

build: move index.ts to operators.ts to avoid node module resolution … #3025

Merged
merged 1 commit into from
Nov 3, 2017

Conversation

jasonaden
Copy link
Collaborator

…in SystemJS

Fixes #2971, #2996, #3011

@vmagalhaes
Copy link

@jasonaden Was there a new version with the fix?

@jasonaden
Copy link
Collaborator Author

@ViictorG1 There will be a new version with this and other fixes, yes. It should end up in 5.5.3.

@rxjs-bot
Copy link

rxjs-bot commented Nov 1, 2017

Messages
📖

CJS: 2278.8KB, global: 746.2KB (gzipped: 120.0KB), min: 145.7KB (gzipped: 31.2KB)

Generated by 🚫 dangerJS

@coveralls
Copy link

coveralls commented Nov 1, 2017

Coverage Status

Coverage remained the same at 97.399% when pulling 759d5c7 on jasonaden:index_to_operators_js into 69bae9b on ReactiveX:stable.

@kwonoj
Copy link
Member

kwonoj commented Nov 1, 2017

/cc @benlesh for second eye.

@benlesh benlesh merged commit d77e3d7 into ReactiveX:stable Nov 3, 2017
@benlesh
Copy link
Member

benlesh commented Nov 3, 2017

Merged and cherry-picked into master as well.

@danwulff
Copy link

danwulff commented Nov 29, 2017

@jasonaden @kwonoj @benlesh any news on when 5.5.3 will be cut?

@lock
Copy link

lock bot commented Jun 6, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 6, 2018
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.

7 participants