-
-
Notifications
You must be signed in to change notification settings - Fork 255
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
ensure we correctly cleanup the context #87
Conversation
stefanpenner
commented
Jul 22, 2015
- pending confirmation in my setup
@@ -7,3 +7,7 @@ export function setContext(context) { | |||
export function getContext() { | |||
return __test_context__; | |||
} | |||
|
|||
export function unsetContext() { |
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.
Why do we need another method here? Could you call 'setContext(undefined)'?
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.
wanted it to be very obvious that It was specifically to unset, not just an overloaded the existing set.
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.
Gotcha, 👍
@stefanpenner 👍 Thanks for dedicating your Mondays to fixing leaks. |
Maybe I should spend Tuesday's learning how to spell:p |
@rwjblue i think this is good to go. |
unsure we correctly cleanup the context
Released in:
|