Enforce braces around conditionals
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
Empieza reproduciendo la entrada de Java con Prettier-Java 1.0.2 y compara la salida actual con la forma esperada con llaves en este issue. Investiga cómo se representan y formatean las instrucciones condicionales y determina después el alcance de exigir llaves de forma coherente. Se considera terminado cuando el ejemplo se formatea con llaves y los casos condicionales relacionados están cubiertos por pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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");
}
}
}
- Lenguaje dominante
- Java
- Estrellas
- 1.2k
- Forks
- 123
- Merge medio
- 2 d 10 h
- PR fusionados (30 d)
- 35
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de jhipster/prettier-java
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
jhipster/prettier-java#1113 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 56/100
jhipster/prettier-java#1114 ·
Los mantenedores suelen responder en 2 días
-
Text block containing a comment-like line with " or \ fails with "Comment was not printed"Posiblemente ocupada @sean0407 la tomó hace 2 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 66/100
jhipster/prettier-java#1112 ·
Los mantenedores suelen responder en 2 días
-
JEP 401 Support value keywordAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
jhipster/prettier-java#1031 ·
Los mantenedores suelen responder en 2 días
-
Adds needlessly parenthesesPosiblemente ocupada @braxtonzhangio la tomó hace 75 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 67/100
jhipster/prettier-java#1001 ·
Los mantenedores suelen responder en 2 días
Todos los issues de jhipster/prettier-java
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
2026 Error Report
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
triplea-game/triplea#15096 ·
Los mantenedores suelen responder en 1 día
-
Implement JDBC Domain Enums (`SqlCategory`, `SqlExecutionStatus`, `SqlExecutionType`) in Core ModuleAbiertocore-module enhancement good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 87/100
sdlc-pro/spring-lens#531 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Low-Drag-MC/LDLib2#78 ·
-
[GLOBAL] Advancement resetAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Vakore/ZappierGames#184 ·