Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Enforce braces around conditionals

Offen
  • Stimmt etwas nicht?
#459 0 Kommentare 25 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
35/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
java
Bereich
tooling

Rechercherichtung

Beginne damit, die Java-Eingabe mit Prettier-Java 1.0.2 zu reproduzieren, und vergleiche die aktuelle Ausgabe mit der erwarteten Form mit geschweiften Klammern in diesem Issue. Untersuche, wie bedingte Anweisungen dargestellt und formatiert werden, und bestimme dann den Umfang, in dem geschweifte Klammern konsistent erzwungen werden sollen. Als abgeschlossen gilt die Aufgabe, wenn das Beispiel mit geschweiften Klammern formatiert wird und verwandte bedingte Fälle durch Tests abgedeckt sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

area: enhancement :wrench:

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");
    }
  }
}
Vorherrschende Sprache
Java
Sterne
1.2k
Forks
123
Ø Merge
2 T. 10 Std.
Gemergte PRs (30 T.)
35

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus jhipster/prettier-java

Alle Issues in jhipster/prettier-java

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.