Reject invalid directives on scalars in 15.0.0
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 85/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- graphql, kotlin
- Área
- backend-api-design
Línea de trabajo
Empieza en SchemaParser#createScalar e inspecciona cómo se registran y omiten actualmente las directivas escalares no válidas. Actualiza DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directives para que compruebe un SchemaError y, después, añade el cambio a las notas de la versión 15.0.0. El trabajo estará terminado cuando la prueba del error escalar y las notas de la versión reflejen el nuevo comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Since #864, scalars get the directives from the schema. When one of them is undeclared or has an argument its definition doesn't have, tools logs Scalar 'Date' is built without its directives from the schema: … and builds the scalar without them, because those scalars always built before. Every other type throws a SchemaError for the same mistake.
What to change
- throw the
SchemaErrorinSchemaParser#createScalarinstead of logging it - turn
DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directivesinto a test for the error - mention it in the 15.0.0 release notes
- Lenguaje dominante
- Kotlin
- Estrellas
- 824
- Forks
- 175
- Merge medio
- 2 d 5 h
- PR fusionados (30 d)
- 52
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.
Más de graphql-java-kickstart/graphql-java-tools
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Los mantenedores suelen responder en 1 día
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Abiertoenhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
graphql-java-kickstart/graphql-java-tools#881 ·
Los mantenedores suelen responder en 1 día
-
Directive wirings see a type reference on recursive input fieldsPosiblemente ocupada @oryan-block la tomó hoy. Abiertobug
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
graphql-java-kickstart/graphql-java-tools#875 ·
Los mantenedores suelen responder en 1 día
Todos los issues de graphql-java-kickstart/graphql-java-tools
Issues similares
-
area/kotlin
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
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
-
good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
dafonton77-tech/Cadreenf#11 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Hy4ri/hermes-mobile#1647 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
rawnaldclark/Stash#649 ·
Los mantenedores suelen responder en 1 día