Skip to content

Commit

Permalink
simplify types a bit further
Browse files Browse the repository at this point in the history
  • Loading branch information
yaacovCR committed Dec 8, 2023
1 parent 6cfc042 commit 296b522
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/execution/buildFieldPlan.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ export type DeferUsageSet = ReadonlySet<DeferUsage>;

export interface FieldGroup {
fields: ReadonlyArray<FieldDetails>;
targets: TargetSet;
knownTargets: TargetSet;
targets?: TargetSet | undefined;
knownTargets?: TargetSet | undefined;
}

export type GroupedFieldSet = Map<string, FieldGroup>;
Expand Down
4 changes: 1 addition & 3 deletions src/execution/execute.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ import type {
GroupedFieldSet,
NewGroupedFieldSetDetails,
} from './buildFieldPlan.js';
import { buildFieldPlan, NON_DEFERRED_TARGET_SET } from './buildFieldPlan.js';
import { buildFieldPlan } from './buildFieldPlan.js';
import type { DeferUsage, FieldDetails } from './collectFields.js';
import { collectFields, collectSubfields } from './collectFields.js';
import type {
Expand Down Expand Up @@ -958,8 +958,6 @@ function getStreamUsage(
node: fieldDetails.node,
deferUsage: undefined,
})),
targets: NON_DEFERRED_TARGET_SET,
knownTargets: NON_DEFERRED_TARGET_SET,
};

const streamUsage = {
Expand Down

0 comments on commit 296b522

Please sign in to comment.