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
feat(compiler): introduce generated injectors #9053
Conversation
try { | ||
return this.getInjectorMetadata(someType, []); | ||
} catch (e) { | ||
if (e.message.indexOf('No InjectorConfig annotation') !== -1) { |
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.
should this be an instanceof
test ? (add todo)
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.
No, we don't have a special error class here. Same as in maybeGetDirectiveMetadata
...
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.
No, we don't have a special error class here
The question is should we ?
ccccd66
to
fd75b27
Compare
This commit is just adding them, but not using them yet for bootstrap. Related to angular#8997
}); | ||
|
||
it('should support property providers', | ||
() => { var inj = SomeConfigInjectorFactory.create(null, new SomeConfig()); }); |
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.
Missing ing.get
??
We won't do this for now. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
First part that only adds features but does not use them yet.