Skip to content

SR-10241 Consolidate StringInterpolationExpr to StringLiteralExpr in Syntax #24718

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
merged 7 commits into from
May 14, 2019

Conversation

kitasuke
Copy link
Contributor

@kitasuke kitasuke commented May 12, 2019

Description

This PR consolidate StringInterpolationExpr to StringLiteralExpr in syntax tree to provide a way to distinguish quotes and string. See more details in the ticket below.

  • Rename StringInterpolationSegments to StringLiteralSegments
  • Rename StringInterpolationExpr to StringLiteralExpr

Ticket

Resolves SR-10241.

Dependency

This PR for SwiftSyntax is also required to apply this change.
swiftlang/swift-syntax#123

@rintaro
Copy link
Member

rintaro commented May 13, 2019

swiftlang/swift-syntax#123
@swift-ci Please test

Copy link
Member

@rintaro rintaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting close!

@kitasuke kitasuke force-pushed the SR-10241_string_literal branch from 3c08fdd to e09a341 Compare May 13, 2019 23:53
@kitasuke
Copy link
Contributor Author

Thank you for your feedbacks again! Addressed them now.

@rintaro
Copy link
Member

rintaro commented May 14, 2019

swiftlang/swift-syntax#123
@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9155ed8

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9155ed8

@rintaro rintaro merged commit 0283e4d into swiftlang:master May 14, 2019
@kitasuke kitasuke deleted the SR-10241_string_literal branch August 3, 2019 04:39
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.

3 participants