jhipster / jhipster/prettier-java

Enforce braces around conditionals

Aperta
#459 0 commenti 25 reazioni 0 assegnatari Vedi su GitHub
area: enhancement :wrench:
Lingua principale
Java
Stelle
1.2k
Fork
120
Merge medio
1g 14h
PR unite (30g)
27

Descrizione

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");
}
}
}
```

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.