prettier-java icon indicating copy to clipboard operation
prettier-java copied to clipboard

Enforce braces around conditionals

Open jhaber opened this issue 3 years ago • 0 comments

Wanted to get thoughts on prettier enforcing braces around conditionals (example below). I believe prettier-js has shot down this request in the past on the grounds that they don't want to modify the AST, but I'm not sure if prettier-java has already crossed that bridge. It would probably be a controversial decision and is maybe a slippery slope, but I think it's pretty well-established at this point that using braces around conditionals is a best practice for correctness and maintainability, so I think it would be reasonable for prettier-java to enforce it.

*Prettier-Java 1.0.2

# Options (if any):
--print-width 80

Input:

public class Test {

  public void method() {
    if (true) System.out.println("hi");
  }
}

Output:

public class Test {

  public void method() {
    if (true) System.out.println("hi");
  }
}

Expected behavior:

public class Test {

  public void method() {
    if (true) {
      System.out.println("hi");
    }
  }
}

jhaber avatar Mar 25 '21 18:03 jhaber