Remove deprecated missingResolverDataFetcher option in 15.0.0
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 65/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- graphql, kotlin
- Domínio
- backend-api-design
Direção de pesquisa
Read SchemaParserOptions, MissingFieldResolver, and FieldResolverScanner to trace the deprecated option through its constructor, builder, and fallback checks. Then inspect MissingFieldResolverTest and remove the test for that option. Done means those paths no longer use missingResolverDataFetcher while missingResolverDataFetcherProvider remains supported; the issue also lists two other breaking cleanups for 15.0.0.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
SchemaParserOptions.missingResolverDataFetcher has been deprecated since 13.0.4 (8b27ed6, March 2023) in favour of missingResolverDataFetcherProvider. We should remove it in the next major release (15.0.0).
It's a breaking change, so it can't go into a 14.x release: SchemaParserOptions.Builder.missingResolverDataFetcher(...) and the SchemaParserOptions.missingResolverDataFetcher property are both public.
What to remove
missingResolverDataFetcherfrom theSchemaParserOptionsconstructor and the builder- the fallback to it in
MissingFieldResolverand the null check inFieldResolverScanner.missingFieldResolver - the test in
MissingFieldResolverTestthat covers the deprecated option
Migration
// before
.missingResolverDataFetcher(fetcher)
// after
.missingResolverDataFetcherProvider { _, _ -> fetcher }
Other breaking cleanups for the same release
- the
environment.getContext()fallback for custom context classes inMethodFieldResolver(already has a "remove in next major" TODO) - the
getContext()fallback for aCoroutineScopepassed as the legacy context object inUtils.coroutineScope(); it now logs a warning and should be read fromGraphQLContextinstead
- Linguagem predominante
- Kotlin
- Estrelas
- 824
- Forks
- 175
- Merge médio
- 2d 5h
- PRs com merge (30d)
- 52
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de graphql-java-kickstart/graphql-java-tools
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 74/100
graphql-java-kickstart/graphql-java-tools#896 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Mantenedores costumam responder em até 1 dia
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Abertaenhancement
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de graphql-java-kickstart/graphql-java-tools
Issues semelhantes
-
enhancement impact:U1 P3 reach:R2 size:S
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 73/100
ankidroid/Anki-Android#22677 ·
Mantenedores costumam responder em até 1 dia
-
Type Pick Request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
reactwg/react-native-releases#1456 · 1 comentário ·
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
yschimke/homeassistant-remotecompose#1038 ·
Mantenedores costumam responder em até 1 dia
-
area:data-quality importer size:S
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
enorm-labs/event-junkie#3707 ·
Mantenedores costumam responder em até 1 dia