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

RemoveScala3OptionalBraces: handle infix on rbrace #3576

Merged
merged 2 commits into from
Jul 7, 2023
Merged

RemoveScala3OptionalBraces: handle infix on rbrace #3576

merged 2 commits into from
Jul 7, 2023

Commits on Jul 6, 2023

  1. Add tests with opt-braces rewrite and new anon

    @kitbellew
    kitbellew committed Jul 6, 2023
    Configuration menu
    Copy the full SHA
    d999ed8 View commit details
    Browse the repository at this point in the history

  2. RemoveScala3OptionalBraces: handle infix on rbrace 

    Instead of handling a specific owner of the left brace (previously, it
was a Term.Match only), let's instead look for infix when we encounter
the right brace instead, and ensure that the next token is not the op
of an infix.

This also handles the case when Term.Match was not an LHS but in the
middle of an infix chain.
    @kitbellew
    kitbellew committed Jul 6, 2023
    Configuration menu
    Copy the full SHA
    e6f1aea View commit details
    Browse the repository at this point in the history