You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes, I think you are right.
The current implementation explicitly checks for a CallExpression of done or its direct parent, so passing done to a function should work but not if it is nested in other expressions like an object literal.
Maybe we should just relax the rule and treat any read of the done callback as "handled".
I agree with you, the code excerpt should not cause an error.
We do lose the "assurance" that the function is at least called at some point in the code, but that did not make sure that the callback was correctly triggered anyway. Just checking the presence of an Identifier with the same name as the callback would be fine IMO.
this should not cause an error:
should it?
The text was updated successfully, but these errors were encountered: