Allow spotless greclipse.properties to be selected via configuration
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- groovy, java
- Área
- build-system, tooling
Línea de trabajo
Comienza examinando testlib/src/main/resources/groovy/greclipse/format/greclipse.properties y el punto de entrada de la configuración del formateador greclipse. Compara las propiedades de Spotless referenciadas con los ajustes revisados en el issue y determina cómo se puede seleccionar una configuración con nombre. Se considera terminado cuando los proyectos pueden elegir la configuración sin copiar el archivo de propiedades y el formato resultante coincide con los ajustes documentados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The default formatting configuration of greclipse isn't very standard or desirable. However, spotless configures greclipse.properties in a much more standard way.
For example, compare the default formatting of googleJavaFormat to greclipse. They're very different.
If you Google site:github.com "greclipse.properties", you will find that many projects simply copied this file as defined by spotless.
Instead of having to copy greclipse.properties into every project, it'd be sweet if we could just select this formatting configuration by spotless via configuration, for example:
<greclipse>
<formatterConfiguration>spotless</file> <!-- default value: eclipse? -->
</greclipse>
EDIT:
Unfortunately, I spoke too soon. I didn't realize spotless's greclipse.properties sets the following (I assume for testing):
groovy.formatter.braces.start=next
groovy.formatter.braces.end=same
These settings aren't very typical. I propose the following settings to stay consistent with googleJavaFormat as well as with examples on the Groovy style guide:
groovy.formatter.braces.end=next
groovy.formatter.braces.start=same
groovy.formatter.longListLength=100
groovy.formatter.multiline.indentation=4
groovy.formatter.remove.unnecessary.semicolons=true
org.eclipse.jdt.core.formatter.indent_empty_lines=false
org.eclipse.jdt.core.formatter.tabulation.char=space
org.eclipse.jdt.core.formatter.tabulation.size=2
- Lenguaje dominante
- Java
- Estrellas
- 5.7k
- Forks
- 560
- Merge medio
- 1 d 6 h
- PR fusionados (30 d)
- 49
Preparar el entorno
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 35/100
diffplug/spotless#3067 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 66/100
diffplug/spotless#3040 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
diffplug/spotless#3013 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de diffplug/spotless
Issues similares
-
P2 testing
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
area/core kind/bug status/triage team/core-shared
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
checkstyle/checkstyle#21755 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
spring-projects/spring-integration#11495 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día