Skip to content

deprecate (internal) buildResolveInfo in favor of (internal) ResolveInfo class#4552

Merged
yaacovCR merged 1 commit intographql:16.x.xfrom
yaacovCR:resolve-info-class
Feb 6, 2026
Merged

deprecate (internal) buildResolveInfo in favor of (internal) ResolveInfo class#4552
yaacovCR merged 1 commit intographql:16.x.xfrom
yaacovCR:resolve-info-class

Conversation

@yaacovCR
Copy link
Contributor

@yaacovCR yaacovCR commented Feb 5, 2026

Motivation: ResolveInfo upgraded in v17 to include abortSignal.

Note: Change appreciable only to those using deep imports.

@yaacovCR yaacovCR requested a review from a team as a code owner February 5, 2026 14:10
@yaacovCR yaacovCR added the PR: bug fix 🐞 requires increase of "patch" version number label Feb 5, 2026
@vercel
Copy link

vercel bot commented Feb 5, 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 changed the title deprecates (internal) buildResolveInfo in favor of new (internal) Res… deprecate (internal) buildResolveInfo in favor of (internal) ResolveInfo class Feb 5, 2026
@yaacovCR yaacovCR force-pushed the resolve-info-class branch 2 times, most recently from e714acb to 728dd6e Compare February 5, 2026 14:44
…olveInfo class

(ResolveInfo upgraded in v17 to be lazy and include abortSignal)

Change appreciable only to those using deep imports.
@yaacovCR yaacovCR merged commit 4149722 into graphql:16.x.x Feb 6, 2026
19 of 20 checks passed
@yaacovCR yaacovCR deleted the resolve-info-class branch February 6, 2026 12:43
yaacovCR added a commit to yaacovCR/graphql-js that referenced this pull request Feb 10, 2026
…ResolveInfo class (graphql#4552)"

See graphql#4553, the lazy ResolveInfo class would break users who wrap resolvers by spreading info.
yaacovCR added a commit that referenced this pull request Feb 11, 2026
…ResolveInfo class (#4552)" (#4555)

See #4553, the lazy ResolveInfo class would break users who wrap resolvers by spreading the `info` argument.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: bug fix 🐞 requires increase of "patch" version number

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant