Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove support for inline class. #1247

Merged
merged 2 commits into from
Aug 10, 2023
Merged

Remove support for inline class. #1247

merged 2 commits into from
Aug 10, 2023

Conversation

munificent
Copy link
Member

That isn't the syntax we're going with and this will let the analyzer team change the AST API.

Fix #1241.

  • Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

cc @oprypin

@munificent
Remove support for inline class.
d77db00 
That isn't the syntax we're going with and this will let the analyzer
team change the AST API.

Fix #1241.
@munificent munificent mentioned this pull request Aug 10, 2023
Closed
@munificent munificent merged commit 0903be1 into main Aug 10, 2023
7 checks passed
@munificent munificent deleted the remove-inline-class branch August 10, 2023 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scheglov scheglov scheglov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Inline classes
2 participants
@munificent @scheglov