| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
PR graphql-java#4229 removed `clearDirectives()` from `GraphQLSchema.Builder` as part of unifying built-in directive handling. However, as noted in issue graphql-java#4259, this method is useful when using `GraphQLSchema.transform` to replace all non-built-in directives: the typical pattern is to collect the non-built-in directives from the source schema, call `clearDirectives()` on the builder, then add back transformed versions of those directives. The new implementation clears `additionalDirectives` entirely. Built-in directives are always re-added automatically at build time by `ensureBuiltInDirectives()`, so they cannot be permanently removed through this method. This treats all built-ins uniformly, consistent with the approach introduced in graphql-java#4229. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Thanks |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
PR #4229 removed clearDirectives() from GraphQLSchema.Builder as part of unifying built-in directive handling. As noted in issue #4259, this method is useful in practice: a common pattern when using GraphQLSchema.transform is to collect the non-built-in directives from the source schema, call clearDirectives() on the builder, and then add back transformed versions of those directives.
This PR re-adds clearDirectives() with semantics that are consistent with the approach introduced in #4229:
Changes
Related
Fixes #4259. See also #4229.