Skip to content

[5.9] Make CodeBlockItemSyntax expressible by string #1740

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 1 commit into from
Jun 8, 2023

Conversation

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Jun 7, 2023

  • Explanation: This commit adds SyntaxParseable conformance for CodeBlockItemSyntax. This is needed to be able to do eg. let item: CodeBlockItemSyntax = "let foo = 1" which is very useful for code block macros. While these aren't enabled in 5.9, cherry-picks will often contain tests for them (with code item macros enabled).
  • Risk: Very low, just adds a public parse for CodeBlockItemSyntax.
  • Original PR: Make CodeBlockItemListSyntax expressible by string literal #1426

@bnbarham bnbarham requested a review from DougGregor June 7, 2023 19:26
@bnbarham bnbarham requested a review from ahoppen as a code owner June 7, 2023 19:26
@bnbarham
Copy link
Contributor Author

bnbarham commented Jun 7, 2023

@swift-ci please test

@kimdv
Copy link
Contributor

kimdv commented Jun 7, 2023

@swift-ci please test windows

@bnbarham
Copy link
Contributor Author

bnbarham commented Jun 7, 2023

swiftlang/swift#66439

@swift-ci please test Windows

@bnbarham
Copy link
Contributor Author

bnbarham commented Jun 7, 2023

Ah that's not going to work still, CI checks out main and then runs update-checkout :(

@shahmishal shahmishal merged commit 7be636b into swiftlang:release/5.9 Jun 8, 2023
@bnbarham bnbarham deleted the cherry-code-item-parse branch June 8, 2023 03:09
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.

6 participants