-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
fix(ivy): ensure module imports are instantiated before the module being declared #35172
Conversation
7798b8f
to
5539ca6
Compare
packages/core/src/di/r3_injector.ts
Outdated
@@ -330,6 +326,9 @@ export class R3Injector { | |||
} | |||
} | |||
} | |||
// Track the InjectorType and add a provider for it. |
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.
Add a comment here about the ordering being important - recursing into children and then adding the injectortype to the set.
@@ -77,6 +77,25 @@ describe('NgModule', () => { | |||
}); | |||
}); | |||
|
|||
it('initializes the module imports before the module itself', () => { |
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.
This test is great, but it would be nice to have a simple r3_injector_spec test as well.
791a4bb
to
865f9ee
Compare
presubmit (failing tests passed in a rerun) |
…ing declared (angular#35172) PR Close angular#35172
…ing declared (angular#35172) PR Close angular#35172
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. |
No description provided.