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

Fixes Rxjs 6 compatibility #5 #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

berga
Copy link

@berga berga commented Jun 9, 2018

I've changed a bit the package requirements:

  • electron updated to ^2.0.2
  • rxjs updated to ^6.2.0

tests have been fixed and run successfully

I've updated the package version to reflect rxjs semver

@colinskow
Copy link
Owner

Thank you very much for this! I hadn't been checking my notifications and just noticed. I will review and merge shortly.

@alexstrat
Copy link

👋 @colinskow do you have a chance to review and publish this? :)

@beorn
Copy link

beorn commented Nov 22, 2020

Any update? This fork also contains some nice improvements: https://github.com/getstation/rx-ipc-electron

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants