Skip to content

refactor: add executionMode/serially argument to executeRootSelectionSet#4704

Merged
yaacovCR merged 1 commit intographql:17.x.xfrom
yaacovCR:serially-root-selection-set
May 1, 2026
Merged

refactor: add executionMode/serially argument to executeRootSelectionSet#4704
yaacovCR merged 1 commit intographql:17.x.xfrom
yaacovCR:serially-root-selection-set

Conversation

@yaacovCR
Copy link
Copy Markdown
Contributor

@yaacovCR yaacovCR commented May 1, 2026

Motivation:

  • more closely matches spec.
  • allows executeSubscriptionEvent to set without re-checking operation
  • drops comment/handling regarding subscribe/execute integration which may never happen.

For now, the argument is only within the Executor, so that non-spec compliant combinations of operations and execution mode are disallowed.

Motivation:
more closely matches spec.
allows executeSubscriptionEvent to set without re-checking operation
drops comment/handling regarding subscribe/execute integration which may never happen.

For now, the argument is only within the Executor, so that non-spec compliant combinations of operations and execution mode are disallowed.
@yaacovCR yaacovCR added the PR: polish 💅 PR doesn't change public API or any observed behaviour label May 1, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 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 merged commit cf0de41 into graphql:17.x.x May 1, 2026
22 of 23 checks passed
@yaacovCR yaacovCR deleted the serially-root-selection-set branch May 1, 2026 09:44
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