Skip to content

Scala: fix parsing of optional semicolon in for loops#7637

Merged
greg-at-moderne merged 1 commit into
mainfrom
greg-scala-semicolon-for-comprehension
May 11, 2026
Merged

Scala: fix parsing of optional semicolon in for loops#7637
greg-at-moderne merged 1 commit into
mainfrom
greg-scala-semicolon-for-comprehension

Conversation

@greg-at-moderne
Copy link
Copy Markdown
Contributor

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

What's changed?

Fix parsing of optional semicolon in for loops. And the variant of it without a semicolon, but a newline in between, e.g.

                  for {
                    _ <- Some(1)
                    field <- Some(2)
                  }

What's your motivation?

They suffer from idempotency issues.

@greg-at-moderne greg-at-moderne self-assigned this May 11, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite May 11, 2026
@greg-at-moderne greg-at-moderne marked this pull request as ready for review May 11, 2026 08:58
@greg-at-moderne greg-at-moderne merged commit 0c6f65e into main May 11, 2026
1 check passed
@greg-at-moderne greg-at-moderne deleted the greg-scala-semicolon-for-comprehension branch May 11, 2026 08:58
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant