Skip to content

Conversation

@feross
Copy link

@feross feross commented Dec 31, 2013

Trivial change, makes the library work when this is not actually window or global. This happens with the popular build tool browserify.

@feross
Copy link
Author

feross commented Jan 26, 2014

Ping. Let's merge this!

@wintermutt
Copy link

+1

@feross
Copy link
Author

feross commented Jan 31, 2014

Yo @madrobby - this is trivial, please merge.

@feross
Copy link
Author

feross commented Feb 6, 2014

Hey @madrobby, just ran into this bug again. This time with chrome apps. Please merge this fix!

@pgte
Copy link

pgte commented Mar 24, 2014

+1, I need this

@andrehaveman
Copy link

+1

@QuentinFchx
Copy link

@madrobby this is definitely required for browserify.

@feross
Copy link
Author

feross commented Oct 7, 2014

@madrobby - Why don't you merge this?

You use global.key throughout this whole module, then suddenly just key at the end. This PR improves the consistency of your code, would make tons of people happy, and costs you nothing. What gives?

@feross
Copy link
Author

feross commented Oct 7, 2014

Nevermind, I see this was fixed in 0f09fc1. Closing.

@feross feross closed this Oct 7, 2014
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.

5 participants