Case sensitive Java type annotations
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza reproduciendo el formato del fragmento de Java con Spotless 2.38.0 en Maven 3.8 y, después, sigue la correspondencia codificada de forma fija de las anotaciones de tipo que utiliza el formateador. Se considera terminado cuando las anotaciones que solo difieren en mayúsculas y minúsculas, como Checker Framework @min y Jakarta Validation @Min, no se tratan como la misma anotación y los ejemplos mostrados se formatean de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We are running Spotless 2.38.0 with Maven 3.8 to format Java and its annotations. Since currently hardcoded list of type annotations doesn't have fully qualified names I think annotations should be matched by case sensitive check.
Case in point, @min annotation from Checker Framework and @Min(X) annotation from Jakarta Validation are two different annotations.
Consider such code:
@NotNull
@Min(60)
@Max(86400)
public long getExpiration() {
return super.getExpiration();
}
Current behaviour results in @Min annotation moved inline with @NotNull and @Max left alone. What's more interesting (and could be related), is that @NotNull and @Min is never moved inline with public long even if I completely remove @Max.
@NotNull @Min(60)
@Max(86400)
public long getExpiration() {
return super.getExpiration();
}
Of source, checking by fully qualified names would be even better solution, but I think case sensitivity would solve current issue in a more trivial way.
- Lenguaje dominante
- Java
- Estrellas
- 5.7k
- Forks
- 560
- Merge medio
- 1 d 9 h
- PR fusionados (30 d)
- 40
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 diffplug/spotless
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 66/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
Todos los issues de diffplug/spotless
Issues similares
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
inu-appcenter/memorIN-backend#288 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
frontend maui-pilot pilot-ask question
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Abiertoarea/plugin
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
kestra-io/plugin-kestra#190 ·