Skip to content

extract IncrementalExecutor from Executor#4545

Merged
yaacovCR merged 1 commit intographql:nextfrom
yaacovCR:next-next
Feb 3, 2026
Merged

extract IncrementalExecutor from Executor#4545
yaacovCR merged 1 commit intographql:nextfrom
yaacovCR:next-next

Conversation

@yaacovCR
Copy link
Contributor

@yaacovCR yaacovCR commented Feb 1, 2026

such that IncrementalExecutor extends a base Executor class that ignores the incremental directives.

In contrast, the ExecutorThrowingOnIncremental subclass should throw on encountering incremental directives, to be used in the standard execute function, signaling that these directives are disabled unless used intentionally with the new experimental behavior via the new function.

@yaacovCR yaacovCR requested a review from a team as a code owner February 1, 2026 13:59
@yaacovCR yaacovCR added the PR: polish 💅 PR doesn't change public API or any observed behaviour label Feb 1, 2026
@vercel
Copy link

vercel bot commented Feb 1, 2026

@yaacovCR is attempting to deploy a commit to the The GraphQL Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@yaacovCR yaacovCR force-pushed the next-next branch 8 times, most recently from 8ccc6a3 to 784db5b Compare February 3, 2026 08:13
such that IncrementalExecutor extends a base Executor class that ignores the incremental directives. In contrast, the ExecutorThrowingOnIncremental subclass should throw on encountering incremental directives, to be used in the standard `execute` function, signalling that these directives are deprecated unless used intentionally with the new experimental behavior via the new function.
@yaacovCR yaacovCR merged commit c4efead into graphql:next Feb 3, 2026
15 of 16 checks passed
@yaacovCR yaacovCR deleted the next-next branch February 3, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: polish 💅 PR doesn't change public API or any observed behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant