Correct documentation for type comparators

leebyron · leebyron · commit 63b9f15e4a61 · 2016-09-28T17:27:38.000-07:00
diff --git a/src/utilities/typeComparators.js b/src/utilities/typeComparators.js
@@ -63,14 +63,14 @@ export function isTypeSubTypeOf(
     return true;
   }
 
-  // If superType is non-null, maybeSubType must also be nullable.
+  // If superType is non-null, maybeSubType must also be non-null.
   if (superType instanceof GraphQLNonNull) {
     if (maybeSubType instanceof GraphQLNonNull) {
       return isTypeSubTypeOf(schema, maybeSubType.ofType, superType.ofType);
     }
     return false;
   } else if (maybeSubType instanceof GraphQLNonNull) {
-    // If superType is nullable, maybeSubType may be non-null.
+    // If superType is nullable, maybeSubType may be non-null or nullable.
     return isTypeSubTypeOf(schema, maybeSubType.ofType, superType);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -63,14 +63,14 @@ export function isTypeSubTypeOf(`
`63`	`63`	`return true;`
`64`	`64`	`}`
`65`	`65`
`66`		`- // If superType is non-null, maybeSubType must also be nullable.`
	`66`	`+ // If superType is non-null, maybeSubType must also be non-null.`
`67`	`67`	`if (superType instanceof GraphQLNonNull) {`
`68`	`68`	`if (maybeSubType instanceof GraphQLNonNull) {`
`69`	`69`	`return isTypeSubTypeOf(schema, maybeSubType.ofType, superType.ofType);`
`70`	`70`	`}`
`71`	`71`	`return false;`
`72`	`72`	`} else if (maybeSubType instanceof GraphQLNonNull) {`
`73`		`- // If superType is nullable, maybeSubType may be non-null.`
	`73`	`+ // If superType is nullable, maybeSubType may be non-null or nullable.`
`74`	`74`	`return isTypeSubTypeOf(schema, maybeSubType.ofType, superType);`
`75`	`75`	`}`
`76`	`76`