-
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): do not throw for invalid metadata #13261
Conversation
84eb399
to
5499591
Compare
this._reportError( | ||
new Error( | ||
`Unexpected value '${stringify(importedType)}' imported by the module '${stringify(moduleType)}'`), | ||
moduleType); |
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.
return
? (in case some code is added below)
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.
Done
new Error( | ||
`Unexpected value '${stringify(type)}' used in the bootstrap property of module '${stringify(moduleType)}'`), | ||
moduleType); | ||
return; |
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.
inside a forEach
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.
The return
is intentional to avoid the push
below.
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.
Yep seen that after (I first thought you wanted to break the loop)
LGTM with a few minor comments |
Hi @chuckjaz, Can you rebase? |
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. |
What kind of change does this PR introduce? (check one with "x")
What is the current behavior? (You can also link to an open issue here)
The language service will sometimes throw with invalid metadata.
#13255
What is the new behavior?
The language service no longer throws.
Does this PR introduce a breaking change? (check one with "x")
Other information:
Fixes #13255