Skip to content

Scala: introduce ExpressionStatement#7606

Merged
greg-at-moderne merged 2 commits intomainfrom
greg-scala-expression-as-statement-in-template-body
May 8, 2026
Merged

Scala: introduce ExpressionStatement#7606
greg-at-moderne merged 2 commits intomainfrom
greg-scala-expression-as-statement-in-template-body

Conversation

@greg-at-moderne
Copy link
Copy Markdown
Contributor

@greg-at-moderne greg-at-moderne commented May 8, 2026

What's changed?

Adding the ExpressionStatement wrapper LST element to Scala.
And using it in several cases.

What's your motivation?

Parsing suffers from idempotency issues. Often swallowing silently some portions of code.

@greg-at-moderne greg-at-moderne marked this pull request as ready for review May 8, 2026 16:28
@greg-at-moderne greg-at-moderne merged commit a6cf1e1 into main May 8, 2026
1 check passed
@greg-at-moderne greg-at-moderne deleted the greg-scala-expression-as-statement-in-template-body branch May 8, 2026 16:28
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant