Skip to content

Commit

Permalink
Remove dev asserts added temporary for v15 to v16 migration
Browse files Browse the repository at this point in the history
  • Loading branch information
IvanGoncharov committed May 11, 2022
1 parent 72090bc commit 167b095
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 20 deletions.
6 changes: 0 additions & 6 deletions src/execution/execute.ts
Original file line number Diff line number Diff line change
Expand Up @@ -165,12 +165,6 @@ export interface ExecutionArgs {
* a GraphQLError will be thrown immediately explaining the invalid input.
*/
export function execute(args: ExecutionArgs): PromiseOrValue<ExecutionResult> {
// Temporary for v15 to v16 migration. Remove in v17
devAssert(
arguments.length < 2,
'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
);

const { schema, document, variableValues, rootValue } = args;

// If arguments are missing or incorrect, throw an error.
Expand Down
7 changes: 0 additions & 7 deletions src/execution/subscribe.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import { devAssert } from '../jsutils/devAssert';
import { inspect } from '../jsutils/inspect';
import { isAsyncIterable } from '../jsutils/isAsyncIterable';
import type { Maybe } from '../jsutils/Maybe';
Expand Down Expand Up @@ -52,12 +51,6 @@ import { getArgumentValues } from './values';
export async function subscribe(
args: ExecutionArgs,
): Promise<AsyncGenerator<ExecutionResult, void, void> | ExecutionResult> {
// Temporary for v15 to v16 migration. Remove in v17
devAssert(
arguments.length < 2,
'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
);

const {
schema,
document,
Expand Down
7 changes: 0 additions & 7 deletions src/graphql.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import { devAssert } from './jsutils/devAssert';
import { isPromise } from './jsutils/isPromise';
import type { Maybe } from './jsutils/Maybe';
import type { PromiseOrValue } from './jsutils/PromiseOrValue';
Expand Down Expand Up @@ -91,12 +90,6 @@ export function graphqlSync(args: GraphQLArgs): ExecutionResult {
}

function graphqlImpl(args: GraphQLArgs): PromiseOrValue<ExecutionResult> {
// Temporary for v15 to v16 migration. Remove in v17
devAssert(
arguments.length < 2,
'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
);

const {
schema,
source,
Expand Down

0 comments on commit 167b095

Please sign in to comment.