Skip to content

chore: forward-port directives on directive definitions (#4521)#4725

Merged
yaacovCR merged 1 commit into
graphql:17.x.xfrom
yaacovCR:directives-on-directive-definitions-17
May 8, 2026
Merged

chore: forward-port directives on directive definitions (#4521)#4725
yaacovCR merged 1 commit into
graphql:17.x.xfrom
yaacovCR:directives-on-directive-definitions-17

Conversation

@yaacovCR
Copy link
Copy Markdown
Contributor

@yaacovCR yaacovCR commented May 8, 2026

Forward-ports #4521 to v17.

Forward-ports 16.x.x commit ad9c519 onto the 17.x codebase.

Includes parser, AST, schema extension, validation, and introspection updates for directives on directive definitions, plus branch-specific adaptations (for example kinds_.ts and current AST optional-field conventions).
@yaacovCR yaacovCR added the PR: bug fix 🐞 requires increase of "patch" version number label May 8, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2026

@BoD 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 65fc947 into graphql:17.x.x May 8, 2026
22 of 23 checks passed
@yaacovCR yaacovCR deleted the directives-on-directive-definitions-17 branch May 8, 2026 07:56
yaacovCR added a commit that referenced this pull request May 10, 2026
## v17.0.0-beta.1 (2026-05-10)

#### Breaking Change 💥
* [#4729](#4729) subscribe: replace perEventExecutor with mapSourceToResponseEvent ([@yaacovCR](https://github.com/yaacovCR))
* [#4730](#4730) chore(engines): drop Node 20 support ([@yaacovCR](https://github.com/yaacovCR))
* [#4731](#4731) refactor(execution): extract buildResolveInfo helper ([@yaacovCR](https://github.com/yaacovCR))

#### New Feature 🚀
* [#4733](#4733) feat: support node v26 ([@yaacovCR](https://github.com/yaacovCR))

#### Bug Fix 🐞
* [#4725](#4725) chore: forward-port directives on directive definitions (#4521) ([@yaacovCR](https://github.com/yaacovCR))
* [#4727](#4727) chore: forward-port configuration of the `ofType` introspection depth (#4317) ([@yaacovCR](https://github.com/yaacovCR))

#### Polish 💅
* [#4728](#4728) polish: group FRAGMENT_VARIABLE_DEFINITION with executable defs ([@yaacovCR](https://github.com/yaacovCR))

#### Internal 🏠
<details>
<summary> 3 PRs were merged </summary>

* [#4460](#4460) internal: use node experimental-strip-types instead of ts-node ([@yaacovCR](https://github.com/yaacovCR))
* [#4732](#4732) internal: use node test instead of mocha + c8 ([@yaacovCR](https://github.com/yaacovCR))
* [#4734](#4734) internal: add comment re: rewriteRelativeImportExtensions ([@yaacovCR](https://github.com/yaacovCR))
</details>

#### Committers: 1
* Yaacov Rydzinski ([@yaacovCR](https://github.com/yaacovCR))
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.

2 participants