Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Smallrye GraphQL 1.3.0 #18999

Merged
merged 3 commits into from
Jul 26, 2021
Merged

Upgrade to Smallrye GraphQL 1.3.0 #18999

merged 3 commits into from
Jul 26, 2021

Conversation

phillip-kruger
Copy link
Member

@phillip-kruger phillip-kruger commented Jul 26, 2021

This PR upgrades SmallRye GraphQL to 1.3.0 (see https://github.com/smallrye/smallrye-graphql/releases/tag/1.3.0)

Main fixes:

  • Moved config mapping from runtime to build time
  • Added more mappings from SmallRye to Quarkus config
  • Use pure vert.x for client (no more JAX-RS)

@phillip-kruger phillip-kruger requested a review from jmartisk July 26, 2021 13:18
@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/graphql area/smallrye labels Jul 26, 2021
@phillip-kruger phillip-kruger added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 26, 2021
@phillip-kruger phillip-kruger changed the title Smallrye graphql 1.3.0 Upgrade to Smallrye graphQL 1.3.0 Jul 26, 2021
@phillip-kruger phillip-kruger changed the title Upgrade to Smallrye graphQL 1.3.0 Upgrade to Smallrye GraphQL 1.3.0 Jul 26, 2021
@phillip-kruger phillip-kruger merged commit b43d144 into quarkusio:main Jul 26, 2021
@quarkus-bot quarkus-bot bot added this to the 2.2 - main milestone Jul 26, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 26, 2021
@phillip-kruger phillip-kruger deleted the smallrye-graphql-1.3.0 branch July 26, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/graphql area/smallrye
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants