Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Use builder pattern to construct Add Plugin recipe visitors

Abierto
  • ¿Algo no está bien?
#269 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
70/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
java
Área
cli

Línea de trabajo

Comienza con src/main/java/com/canonical/devpackspring/rewrite/visitors/KotlinAddPluginVisitor.java alrededor de las líneas 42-43 e inspecciona el constructor de AddPluginVisitorSupport y sus otros puntos de llamada. Introduce el enfoque de construcción con nombre descrito en el issue, actualiza los recipe visitors que usan el constructor y verifica que el proyecto siga compilando.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

─── src/main/java/com/canonical/devpackspring/rewrite/visitors/KotlinAddPluginVisitor.java:42-43 ───
[maintainability · low] The `AddPluginVisitorSupport` constructor takes 9 positional parameters, 7
of which are Strings/booleans of easily-confusable types (pluginName, pluginVersion, buildFileName,
and three template strings). At this call site it's easy to accidentally swap e.g.
`PLUGIN_TEMPLATE_KOTLIN` with `BUILT_IN_TEMPLATE_KOTLIN`, or the build file name with a template,
and the compiler won't catch it. Consider introducing a small parameter object or builder (e.g.
`AddPluginVisitorSupport.builder().plugin(...).templates(...).operations(...)`) so each argument is
named at the call site and mis-ordering is impossible.

-               this.support = new AddPluginVisitorSupport<>(pluginName, pluginVersion, subprojects, parser, "build.gradle.kts",
-                               PLUGIN_TEMPLATE_KOTLIN, BUILT_IN_TEMPLATE_KOTLIN, SUBPROJECTS_TEMPLATE_KOTLIN, new KotlinOperations());
+               this.support = AddPluginVisitorSupport.<K.CompilationUnit>builder()
+                       .plugin(pluginName, pluginVersion)
+                       .subprojects(subprojects)
+                       .parser(parser)
+                       .buildFileName("build.gradle.kts")
+                       .templates(PLUGIN_TEMPLATE_KOTLIN, BUILT_IN_TEMPLATE_KOTLIN, SUBPROJECTS_TEMPLATE_KOTLIN)
+                       .operations(new KotlinOperations())
+                       .build();
Lenguaje dominante
Java
Estrellas
1
Forks
5
Merge medio
2 d 30 min
PR fusionados (30 d)
17

Preparar el entorno

Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de canonical/devpack-for-spring-cli

Todos los issues de canonical/devpack-for-spring-cli

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.