Remove `BiMap` and `SchemaParserDictionary.getDictionary()` in 15.0.0
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #908 de @oryan-block — fusionado
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- backend-api-design
Línea de trabajo
Read the changes in #861, then locate SchemaParserDictionary, BiMap, BiMapTest, and SchemaParserDictionaryTest to identify the deprecated API and tests named in the issue. Before removing them, check GraphQLJavaToolsAutoConfiguration in graphql-spring-boot and confirm that dependent release uses toMap(). Done means the listed API and tests are removed and the Spring Boot call site has migrated in a compatible release.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
#861 replaces the library's own use of graphql.kickstart.tools.util.BiMap with plain maps and adds SchemaParserDictionary.toMap(). getDictionary() and BiMap stay for now because they're public, and are deprecated. We should remove both in the next major release (15.0.0).
It's a breaking change, so it can't go into a 14.x release. getDictionary() returns a BiMap, so dropping the class means dropping the method too, and changing its return type in place breaks binary compatibility anyway.
What to remove
SchemaParserDictionary.getDictionary()graphql.kickstart.tools.util.BiMapandBiMapTest- the
getDictionary()test inSchemaParserDictionaryTest
Before removing
graphql-spring-boot calls builder.dictionary(dictionary.getDictionary()) in GraphQLJavaToolsAutoConfiguration. It needs to switch to toMap() in a release that depends on a graphql-java-tools version that has it.
Migration
// before
builder.dictionary(dictionary.getDictionary())
// after
builder.dictionary(dictionary.toMap())
- Lenguaje dominante
- Kotlin
- Estrellas
- 824
- Forks
- 175
- Merge medio
- 1 d 10 h
- PR fusionados (30 d)
- 86
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Issues similares
-
Messages: Subscribe to the new communication websocket topicsPosiblemente ocupada @krusche la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ls1intum/artemis-android#801 ·
Los mantenedores suelen responder en 1 día
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
keiyoushi/extensions-source#20362 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
enhancement good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
software-mansion/enriched-markdown#1084 ·
Los mantenedores suelen responder en 2 días