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

Update: curly multi-or-nest flagging semis on next line (fixes #12370) #12378

Merged
merged 2 commits into from Nov 1, 2019
Merged

Update: curly multi-or-nest flagging semis on next line (fixes #12370) #12378

merged 2 commits into from Nov 1, 2019

Commits on Oct 12, 2019

  1. Update: Curly multi-or-nest flagging semis on next line (fixes #12370)

    @cherryblossom000
    cherryblossom000 committed Oct 12, 2019
    Copy the full SHA
    e7f4603 View commit details
    Browse the repository at this point in the history

  2. Chore: Add more tests to the multi-or-nest option for curly 

    Check that multi-or-nest it fixes cases like
if (foo) {
  doSomething()
  ;
}

and that it ignores cases with an empty statement like
if (foo)
;
    @cherryblossom000
    cherryblossom000 committed Oct 12, 2019
    Copy the full SHA
    62e19fe View commit details
    Browse the repository at this point in the history