-
Notifications
You must be signed in to change notification settings - Fork 3.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
Do not allow arrow functions as the "factory" function for registerServiceDorDoc #26776
Do not allow arrow functions as the "factory" function for registerServiceDorDoc #26776
Conversation
Hey @jridgewell, these files were changed:
|
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.
LGTM
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.
Hang on... I'm summoning my inner @jridgewell as I read this...
Okay, LGTM.
i'm going to file a bug against closure compiler as well, as Arrow Functions should not be allowed to be passed into a parameter that is |
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.
Thanks for addressing so quickly.
…rviceDorDoc (#26776) * don't allow registration of arrow function as factory * apply lint fixes * Do not allow arrow functions as the "factory" function
…rviceDorDoc (#26776) * don't allow registration of arrow function as factory * apply lint fixes * Do not allow arrow functions as the "factory" function
in the module build this errors out because arrow functions are not
newable