Skip to content

[utils/gyb_syntax_support] Add the classification kind for identifiers #25945

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

Merged

Conversation

akyrtzi
Copy link
Contributor

@akyrtzi akyrtzi commented Jul 2, 2019

Explanation: SwiftSyntax’s classification mechanism should tag normal identifier tokens as Identifier. This is useful for community tools using this mechanism from SwiftSyntax. For more context see this post: https://forums.swift.org/t/classification-of-identifier/20860
Scope: Affects the syntax classifier mechanism of SwiftSyntax
Issue: rdar://problem/52543169
Risk: Very Low, doesn't affect anything in the compiler side
Testing: Updated tests in SwiftSyntax repo
Reviewer: Xi Ge (@nkcsgexi)

@akyrtzi
Copy link
Contributor Author

akyrtzi commented Jul 2, 2019

swiftlang/swift-syntax#132

@swift-ci Please test

@akyrtzi
Copy link
Contributor Author

akyrtzi commented Jul 2, 2019

@swift-ci Please nominate

@akyrtzi akyrtzi merged commit 8672d6c into swiftlang:swift-5.1-branch Jul 2, 2019
@akyrtzi akyrtzi deleted the 5.1-syntax-classify-identifiers branch July 2, 2019 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant