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

RedundantBraces: don't remove empty blocks #4148

Merged
merged 3 commits into from
Aug 8, 2024
Merged

RedundantBraces: don't remove empty blocks #4148

merged 3 commits into from
Aug 8, 2024

Commits on Aug 8, 2024

  1. Test dependency redundant/optional braces rewrites

    @kitbellew
    kitbellew committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    8a69ae4 View commit details
    Browse the repository at this point in the history

  2. RedundantBraces: move case to outer match 

    The other invocations of `okToRemoveBlock` it used to be part of never
apply to this circumstance.
    @kitbellew
    kitbellew committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    b1874a5 View commit details
    Browse the repository at this point in the history

  3. RedundantBraces: don't remove empty blocks 

    There are only limited contexts where it is OK (`case` being one, but
it is handled elsewhere).
    @kitbellew
    kitbellew committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    6dc4bf5 View commit details
    Browse the repository at this point in the history