-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Assign to self #52
Assign to self #52
Conversation
If that works for you that works for me :-). Ready when you are… |
@@ -17,6 +17,7 @@ | |||
"eqnull": true, | |||
"browser": true, | |||
"globals": { | |||
"Promise": false | |||
"Promise": false, | |||
"self": false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might make more sense to switch the worker
option on rather than defining self
as a global.
(Similarly but potentially less importantly you could choose to switch on esnext
instead of defining Promise
as a global — but that defines a larger amount of stuff that you might not want defined)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh nice, I didn't realize there was a worker
option.
I'll give exnext
a try too. Last time I think I was on an older jshint that didn't define Promise
yet.
👍 |
isomorphic-fetch
will need to be changed to stubglobal.self
instead ofglobal.window
.https://github.com/matthew-andrews/isomorphic-fetch/blob/master/server.js
Changes #48.
Obsoletes #49, #50.
cc @dgraham @matthew-andrews @knowledgecode