Skip to content

Conversation

rintaro
Copy link
Member

@rintaro rintaro commented Mar 27, 2025

Update for swiftlang/swift-syntax#3028

In SwiftSyntax, some cases in AttributeSyntax.Arguments enum are being consolidated to .argumentList(LabeledExprListSyntax)

@rintaro rintaro marked this pull request as draft March 27, 2025 20:14
@rintaro
Copy link
Member Author

rintaro commented Mar 27, 2025

swiftlang/swift-syntax#3028
@swift-ci Please smoke test

@rintaro rintaro force-pushed the astgen-attributes-reorg branch from 5d0fa4e to 02f2676 Compare March 27, 2025 20:31
In `SwiftSyntax`, some cases in `AttributeSyntax.Arguments` enum are
being consolidated to `.argumentList(LabeledExprListSyntax)`
@rintaro rintaro force-pushed the astgen-attributes-reorg branch from 02f2676 to 2a21e68 Compare March 27, 2025 21:03
@rintaro
Copy link
Member Author

rintaro commented Mar 27, 2025

swiftlang/swift-syntax#3028
@swift-ci Please smoke test

@rintaro
Copy link
Member Author

rintaro commented Mar 28, 2025

@rintaro rintaro marked this pull request as ready for review March 28, 2025 16:42
@rintaro rintaro changed the title [ASTGen] Update for specialied attribute kind removal [ASTGen] Update for specialized attribute kind removal Mar 28, 2025
@rintaro
Copy link
Member Author

rintaro commented Mar 28, 2025

swiftlang/swift-format#971
swiftlang/swift-syntax#3028
@swift-ci Please smoke test Windows

let arg = node.arguments?.as(TokenSyntax.self)
else {
print("Not a token")
// TODO: Diagnose.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this comment be preserved? Here and all the case below

Copy link
Member Author

Choose a reason for hiding this comment

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

generateWithLabeledExprListArguments and generateConsuming* will diagnose if the input is unexpected.

@bnbarham bnbarham merged commit 26259f6 into swiftlang:main Mar 31, 2025
3 checks passed
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