Always invoke GraphQLNonNull as a constructor

This was always the recommended style, but now that `GraphQLNonNull` and `GraphQLList` have been converted to proper classes in graphql/graphql-js#2906, calling them as functions no longer works.
apollographql · Mar 25, 2021 · 1b2af2d · 1b2af2d
1 parent 0b133e5
commit 1b2af2d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/federation-js/src/coreSpec.ts b/federation-js/src/coreSpec.ts
@@ -10,7 +10,7 @@ export const CoreDirective = new GraphQLDirective({
   locations: [DirectiveLocation.SCHEMA],
   args: {
     feature: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
   isRepeatable: true,

diff --git a/federation-js/src/csdlDirectives.ts b/federation-js/src/csdlDirectives.ts
@@ -11,7 +11,7 @@ export const ComposedGraphDirective = new GraphQLDirective({
   locations: [DirectiveLocation.SCHEMA],
   args: {
     version: {
-      type: GraphQLNonNull(GraphQLInt),
+      type: new GraphQLNonNull(GraphQLInt),
     },
   },
 });
@@ -21,10 +21,10 @@ export const GraphDirective = new GraphQLDirective({
   locations: [DirectiveLocation.SCHEMA],
   args: {
     name: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
     url: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
   isRepeatable: true,
@@ -35,7 +35,7 @@ export const OwnerDirective = new GraphQLDirective({
   locations: [DirectiveLocation.OBJECT],
   args: {
     graph: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });
@@ -45,10 +45,10 @@ export const KeyDirective = new GraphQLDirective({
   locations: [DirectiveLocation.OBJECT],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
     graph: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
   isRepeatable: true,
@@ -59,7 +59,7 @@ export const ResolveDirective = new GraphQLDirective({
   locations: [DirectiveLocation.FIELD_DEFINITION],
   args: {
     graph: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });
@@ -69,7 +69,7 @@ export const ProvidesDirective = new GraphQLDirective({
   locations: [DirectiveLocation.FIELD_DEFINITION],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });
@@ -79,7 +79,7 @@ export const RequiresDirective = new GraphQLDirective({
   locations: [DirectiveLocation.FIELD_DEFINITION],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });

diff --git a/federation-js/src/directives.ts b/federation-js/src/directives.ts
@@ -26,7 +26,7 @@ export const KeyDirective = new GraphQLDirective({
   locations: [DirectiveLocation.OBJECT, DirectiveLocation.INTERFACE],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });
@@ -46,7 +46,7 @@ export const RequiresDirective = new GraphQLDirective({
   locations: [DirectiveLocation.FIELD_DEFINITION],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });
@@ -56,7 +56,7 @@ export const ProvidesDirective = new GraphQLDirective({
   locations: [DirectiveLocation.FIELD_DEFINITION],
   args: {
     fields: {
-      type: GraphQLNonNull(GraphQLString),
+      type: new GraphQLNonNull(GraphQLString),
     },
   },
 });