-
Notifications
You must be signed in to change notification settings - Fork 25k
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(language-service): Clear caches when program changes #21337
Conversation
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 code looks completely unnecessary to me and, if it has an effect, there is something more fundamental that is going wrong.
The only way analyzedModules should change is if the content of the program changes. Since validate
will clear analyzedModules
whenever that happens (called before calling ensureAnalyzedModules()
the only way this change would have the desired effect is there was something wrong with validate()
.
This commit fixes a bug whereby the caches are not cleared when the program changes. This subsequently produces the incorrect error of 'Component ... is not included in a module ...'. PR Close angular#19405
I see that you just added the
If you want your PR to be merged, it has to pass all the checks. But if you have a good reason to want to merge this, please contact the caretaker to let them know. |
This commit fixes a bug whereby the caches are not cleared when the program changes. This subsequently produces the incorrect error of 'Component ... is not included in a module ...'. PR Close angular#19405 PR Close angular#21337
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. |
This commit fixes a bug whereby the empty set of analyzed modules is
cached and subsequently produces the incorrect error 'Component ... is
not included in a module...'
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Error message 'Component ... is not included in a module...' is incorrectly shown.
Issue Number: 19405
What is the new behavior?
Language service no longer shows incorrect error message.
Does this PR introduce a breaking change?
Other information