jhipster / jhipster/prettier-java

Enforce braces around conditionals

Open
#459 0 comments 25 reactions 0 assignees View on GitHub
area: enhancement :wrench:
Dominant language
Java
Stars
1.2k
Forks
120
Avg merge
1d 14h
Merged PRs (30d)
27

Description

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*

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

**Input:**

```java
public class Test {

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

**Output:**

```java
public class Test {

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

**Expected behavior:**

```java
public class Test {

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.