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

Make Edge.js always return a browser object #18

Merged
merged 1 commit into from
Nov 10, 2016

Conversation

gkatsev
Copy link
Contributor

@gkatsev gkatsev commented Nov 8, 2016

This makes it line up with the other browser files and fixes #17.

This makes it line up with the other browser files and fixes litixsoft#17.
@fanatid
Copy link

fanatid commented Nov 8, 2016

ACK
please release patch version with this PR

@nickserv
Copy link
Contributor

nickserv commented Nov 9, 2016

Ultimately resolve is a buggy dependency that isn't properly tested on recent Node versions, but this is a great temporary fix and it should be merged as soon as possible.

nickserv referenced this pull request Nov 9, 2016
@LitixThomas LitixThomas merged commit bd36623 into litixsoft:master Nov 10, 2016
@gkatsev gkatsev deleted the patch-1 branch November 10, 2016 16:55
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.

Plugin fails on systems w/o Edge
4 participants