search

apollographql / apollo-kotlin

:rocket:  A strongly-typed, caching GraphQL client for the JVM, Android, and Kotlin multiplatform.
https://www.apollographql.com/docs/kotlin
MIT License
3.73k stars 654 forks source link

Validate repeatable directives #4172

Open martinbonnin opened 2 years ago

martinbonnin commented 2 years ago

Currently it is possible to repeat non-repeatable directives:

{
  field @nonnull @nonnull
}

This should throw a validation error instead

martinbonnin commented 1 month ago

This is not done yet. See https://github.com/apollographql/apollo-kotlin/blob/7668a302eb5c62ca1d3e423ccfefa1142acee1f4/libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ValidationCommon.kt#L91