Repository navigation
Actualiza el SDK Java con Fastjson2, soporte LTS y controles de calidad - #1
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
El RUC completo que devuelve la API, como
80024191-6, fallaba al deserializarse comoInteger. El cliente representa ahora los siete campos de la respuesta y se verifica con Java 17, 21 y 25.Evidence
JsonSyntaxException; los tests dependían delocalhost:8080.After:
looksUpContributorWithFullRucpasa con un servidor local de puerto efímero. Los nueve casos HTTP pasan en cada JDK 17/21/25, incluidos Unicode, paginación, errores HTTP y JSON inválido../gradlew --no-daemon clean build -PjavaVersion=<17|21|25>pasa con formato y PMD de producción/tests. El JAR construido con JDK 25 funciona en un consumidor Java 17; bytecode 61 y dependencias publicadas verificados. Wrapper cotejado con los SHA oficiales.Merge Danger
Door: two-way
El merge es reversible y no publica un artefacto automáticamente.
Blast Radius: Consumidores
La versión 0.1.0 requiere adaptar y recompilar consumidores:
getRuc()/setRuc()usanString, el constructor completo incorpora los siete campos ygetDoc()expone el documento numérico. Java mínimo 17. Los errores del conversor se propagan comoIOExceptioncon causa; deben adaptarse capturas específicas de Gson. JDK 25 muestra un aviso deUnsafede Fastjson2 que permanece visible. No se afirma una mejora de rendimiento medida.