Skip to content

Make DeclarationModifier and TypeAttribute normal TokenSpecSet instead of generating them #1715

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 3, 2023

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented May 30, 2023

There’s no reason to generate these TokenSpecSet anymore.

@ahoppen ahoppen requested a review from bnbarham May 30, 2023 18:37
@ahoppen
Copy link
Member Author

ahoppen commented May 30, 2023

@swift-ci Please test

Copy link
Contributor

@kimdv kimdv left a comment

Choose a reason for hiding this comment

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

Did you run the code generation? 😅

I see two templates removed from the list

@ahoppen ahoppen force-pushed the ahoppen/normal-spec-sets branch from bed03ed to 9273816 Compare May 30, 2023 19:49
@ahoppen
Copy link
Member Author

ahoppen commented May 30, 2023

Oh, yeah. Apparently I missed the deletion of the generated files at some point

@kimdv
Copy link
Contributor

kimdv commented May 30, 2023

@swift-ci please test

@ahoppen
Copy link
Member Author

ahoppen commented May 30, 2023

@swift-ci Please test Windows

@ahoppen ahoppen force-pushed the ahoppen/normal-spec-sets branch from 9273816 to 4233ff6 Compare June 2, 2023 22:10
@ahoppen
Copy link
Member Author

ahoppen commented Jun 2, 2023

@swift-ci Please test

@@ -11,5 +11,5 @@
url: https://forums.swift.org/tags/c/development/tooling/39/swift-syntax
about: Ask and answer questions about SwiftSyntax
- name: SwiftSyntax Documentation
url: https://github.com/apple/swift-syntax/tree/main/Documentation
url: https://swiftpackageindex.com/apple/swift-syntax/documentation/swiftsyntax
Copy link
Contributor

Choose a reason for hiding this comment

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

Didn't you already change this in another PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, rebase mistake

@ahoppen ahoppen force-pushed the ahoppen/normal-spec-sets branch from 4233ff6 to 19c9116 Compare June 2, 2023 22:17
@ahoppen
Copy link
Member Author

ahoppen commented Jun 2, 2023

@swift-ci Please test

@ahoppen ahoppen enabled auto-merge June 2, 2023 22:17
@ahoppen
Copy link
Member Author

ahoppen commented Jun 2, 2023

@swift-ci Please test Windows

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented Jun 2, 2023

@swift-ci Please test Windows

…instead of generating them

There’s no reason to generate these `TokenSpecSet` anymore.
@ahoppen ahoppen force-pushed the ahoppen/normal-spec-sets branch from 19c9116 to f2afad8 Compare June 3, 2023 00:08
@ahoppen
Copy link
Member Author

ahoppen commented Jun 3, 2023

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Jun 3, 2023

@swift-ci Please test Windows

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented Jun 3, 2023

@swift-ci Please test Windows

@ahoppen ahoppen merged commit ae12a23 into swiftlang:main Jun 3, 2023
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