-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Provide all possible keywords in xml doc comment langword
tag
#72694
Conversation
|
||
foreach (var keywordKind in SyntaxFacts.GetKeywordKinds()) | ||
{ | ||
var keywordText = SyntaxFacts.GetText(keywordKind); |
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 is reusing the same logic as the actual product code, which isn't great for a test at all, but I cannot think of a better solution here. Another option is to manually list all keywords we need to check for, but this is an unmanageble nightmare. We sort of want keyword list to be tight to SyntaxFacts.GetKeywordKinds
, so as new keywords are added to the language we immediately test that they are in the list. Same thing on VB side
test failure seems legit. |
@CyrusNajmabadi PTAL |
Thanks! |
Fixes: #72258