kpm - not setting plugin version should possible in PluginNamingResolver
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza con PluginNamingResolver.java, especialmente con su constructor, getPluginVersion() y las fábricas estáticas; después, inspecciona ArtifactAndVersionFinder.java alrededor de la llamada referenciada. Sigue cómo se usa stringContainsVersion y revisa las pruebas existentes del resolver, si las hay. Se considera terminado cuando un pluginVersion omitido puede analizarse a partir del nombre del plugin y las versiones no resueltas siguen el fallback indicado sin producir una IllegalStateException.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
If PluginNamingResolver created without specifying the pluginVersion, getPluginVersion() throw IllegalStateException. It is not correct, since we have a chances to analyzing version from stringContainsVersion.
IIRC, that assertion added when working with ArtifactAndVersionFinder, but as workaround, we can change that line in ArtifactAndVersionFinder with:
final String pluginNameAsArtifactId = pluginKey + "-plugin";
And then, we can remove PluginNamingResolver.of(pluginKey) static factory. And start to re-condider getPluginVersion() implementation. After all, none of operation in PluginNamingResolver should throw an exception, since even if we cannot found any version, it will return 0.0.0.
- Lenguaje dominante
- Java
- Estrellas
- 30
- Forks
- 42
- Merge medio
- 2 d 14 h
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 killbill/killbill-platform
-
bug
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
killbill/killbill-platform#193 ·
-
LogsSseHandler: allow caller to filter logs by accountId and userTokenQuizá libre de nuevo @xsalefter la tomó hace 125 días y no hay ningún pull request abierto. Abiertoenhancement
killbill/killbill-platform#163 · 1 asignado ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
killbill/killbill-platform#160 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
Todos los issues de killbill/killbill-platform
Issues similares
-
enhancement triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ShaftHQ/SHAFT_ENGINE#7902 ·
Los mantenedores suelen responder en 1 día
-
jules
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
open-learning-exchange/myplanet#18568 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
quarkiverse/quarkus-ironjacamar#264 · 7 comentarios ·
-
[Bug]: Login shows only "Log-in failed" when Commons answers the login request with an API errorAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
commons-app/apps-android-commons#7295 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día