Remove `BiMap` and `SchemaParserDictionary.getDictionary()` in 15.0.0
Les mainteneurs répondent en général sous 1 jour
Une pull request liée a déjà été fusionnée.
- #908 par @oryan-block — fusionnée
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 48/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Domaine
- backend-api-design
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
#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())
- Langage dominant
- Kotlin
- Étoiles
- 824
- Forks
- 175
- Merge moyen
- 1 j 10 h
- PR mergées (30 j)
- 86
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Issues similaires
-
staticAssetsPath follows symlinks outside the directory on WindowsPeut-être pris @ybelMekk l’a pris aujourd’hui. Ouvertebug security findings
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
navikt/mock-oauth2-server#1132 ·
Les mainteneurs répondent en général sous 1 jour
-
a:plugin-description
Difficulté 1/5 Moins d'une heure Accessibilité débutants 76/100
gradle/plugin-portal-requests#319 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
SimonHalvdansson/Harmonic-HN#399 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
[Request] 李跳跳Ouverterequest-adaption
Difficulté 2/5 Moins d'une heure Accessibilité débutants 78/100
BetterAndroid/android-notification-icon-project#799 ·
Les mainteneurs répondent en général sous 1 jour