Skip to content

Bump graphql to 24.1#2740

Merged
tsegismont merged 2 commits intovert-x3:masterfrom
yeikel:patch-2
Jun 3, 2025
Merged

Bump graphql to 24.1#2740
tsegismont merged 2 commits intovert-x3:masterfrom
yeikel:patch-2

Conversation

@yeikel
Copy link
Copy Markdown
Contributor

@yeikel yeikel commented May 16, 2025

@tsegismont
Copy link
Copy Markdown
Member

@vietj I believe this should be part of Vert.x 5.0.1.

The GraphQL Java team considers v23.0 poisoned (see the release notes), so users should jump to v24.0 immediately.

If that's fine with you, I'll scope the PR to the correct milestone and merge.

Copy link
Copy Markdown
Member

@tsegismont tsegismont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yeikel Thank you !

@yeikel yeikel changed the title Bump graphql to 24.0 Bump graphql to 24.1 May 30, 2025
@yeikel
Copy link
Copy Markdown
Contributor Author

yeikel commented May 30, 2025

@tsegismont 24.1 was released today. To avoid another subsequent pull request, I updated this one instead. Please re-review and let me know if you have any concerns :)

@vietj
Copy link
Copy Markdown
Contributor

vietj commented May 30, 2025

@tsegismont works for me

@tsegismont tsegismont added this to the 5.1.0 milestone Jun 3, 2025
@tsegismont tsegismont merged commit c363f58 into vert-x3:master Jun 3, 2025
7 checks passed
tsegismont pushed a commit that referenced this pull request Jun 3, 2025
tsegismont added a commit that referenced this pull request Jun 3, 2025
* Bump graphql to 24.0

https://github.com/graphql-java/graphql-java/releases/tag/v24.0

* 24.1

Co-authored-by: Yeikel Santana <email@yeikel.com>
@yeikel yeikel deleted the patch-2 branch June 3, 2025 12:02
tsegismont added a commit to tsegismont/vertx-web that referenced this pull request Oct 3, 2025
* Bump graphql to 24.0

https://github.com/graphql-java/graphql-java/releases/tag/v24.0

* 24.1

Co-authored-by: Yeikel Santana <email@yeikel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants