function overloading

Author: saihaj

src/jsutils/didYouMean.ts

@@ -3,15 +3,15 @@ const MAX_SUGGESTIONS = 5;
 /**
  * Given [ A, B, C ] return ' Did you mean A, B, or C?'.
  */
-declare function didYouMean(suggestions: ReadonlyArray<string>): string;
-// eslint-disable-next-line no-redeclare
-declare function didYouMean(
+export function didYouMean(suggestions: ReadonlyArray<string>): string;
+export function didYouMean(
   subMessage: string,
   suggestions: ReadonlyArray<string>,
 ): string;
-
-// eslint-disable-next-line no-redeclare
-export function didYouMean(firstArg, secondArg) {
+export function didYouMean(
+  firstArg: string | ReadonlyArray<string>,
+  secondArg?: ReadonlyArray<string>,
+) {
   const [subMessage, suggestionsArg] =
     typeof firstArg === 'string'
       ? [firstArg, secondArg]

src/jsutils/isAsyncIterable.ts

@@ -2,12 +2,8 @@
  * Returns true if the provided object implements the AsyncIterator protocol via
  * implementing a `Symbol.asyncIterator` method.
  */
-declare function isAsyncIterable(
-  value: unknown,
-  // $FlowFixMe[invalid-in-rhs]
-): value is AsyncIterable<unknown>;
-
-// eslint-disable-next-line no-redeclare
-export function isAsyncIterable(maybeAsyncIterable) {
+export function isAsyncIterable(
+  maybeAsyncIterable: unknown,
+): maybeAsyncIterable is AsyncIterable<unknown> {
   return typeof maybeAsyncIterable?.[Symbol.asyncIterator] === 'function';
 }

src/jsutils/isIterableObject.ts

@@ -14,13 +14,9 @@
  * isIterableObject({ key: 'value' }) // false
  * isIterableObject({ length: 1, 0: 'Alpha' }) // false
  */
-declare function isIterableObject(
-  value: unknown,
-  // $FlowFixMe[invalid-in-rhs]
-): value is Iterable<unknown>;
-
-// eslint-disable-next-line no-redeclare
-export function isIterableObject(maybeIterable: unknown): boolean {
+export function isIterableObject(
+  maybeIterable: unknown,
+): maybeIteratable is Iterable<unknown> {
   return (
     typeof maybeIterable === 'object' &&
     typeof maybeIterable?.[Symbol.iterator] === 'function'

src/jsutils/toObjMap.ts

@@ -5,12 +5,9 @@ import type {
   ReadOnlyObjMapLike,
 } from './ObjMap';
 
-/* eslint-disable no-redeclare */
-declare function toObjMap<T>(obj: ObjMapLike<T>): ObjMap<T>;
-declare function toObjMap<T>(obj: ReadOnlyObjMapLike<T>): ReadOnlyObjMap<T>;
-
-export function toObjMap(obj) {
-  /* eslint-enable no-redeclare */
+export function toObjMap<T>(obj: ObjMapLike<T>): ObjMap<T>;
+export function toObjMap<T>(obj: ReadOnlyObjMapLike<T>): ReadOnlyObjMap<T>;
+export function toObjMap<T>(obj: ObjMapLike<T> | ReadOnlyObjMapLike<T>) {
   if (Object.getPrototypeOf(obj) === null) {
     return obj;
   }

src/utilities/typeFromAST.ts

@@ -20,21 +20,19 @@ import { GraphQLList, GraphQLNonNull } from '../type/definition';
  * the type called "User" found in the schema. If a type called "User" is not
  * found in the schema, then undefined will be returned.
  */
-/* eslint-disable no-redeclare */
-declare function typeFromAST(
+export function typeFromAST(
   schema: GraphQLSchema,
   typeNode: NamedTypeNode,
-): GraphQLNamedType | void;
-declare function typeFromAST(
+): GraphQLNamedType | undefined;
+export function typeFromAST(
   schema: GraphQLSchema,
   typeNode: ListTypeNode,
-): GraphQLList<any> | void;
-declare function typeFromAST(
+): GraphQLList<any> | undefined;
+export function typeFromAST(
   schema: GraphQLSchema,
   typeNode: NonNullTypeNode,
-): GraphQLNonNull<any> | void;
+): GraphQLNonNull<any> | undefined;
 export function typeFromAST(schema, typeNode) {
-  /* eslint-enable no-redeclare */
   let innerType;
   if (typeNode.kind === Kind.LIST_TYPE) {
     innerType = typeFromAST(schema, typeNode.type);