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

Update to Java 11, Gradle 8.3, GraphQL Java v21.0 #113

Merged
merged 5 commits into from
Aug 21, 2023
Merged

Conversation

dondonz
Copy link
Member

@dondonz dondonz commented Aug 21, 2023

Upgrade PR ahead of the v21 release of this library

  • Java version upgraded from 8 -> 11, in line with change in GraphQL Java v21
  • Builds (GitHub Actions) upgraded to Java 11
  • Upgraded to Gradle 8.3 (latest) so we can take advantage of the Java toolchain feature, which makes version management much easier
  • Upgrade GraphQL Java dependency to v21.0 (no breaking changes affecting this library)
  • Upgrade GitHub Action versions

with:
java-version: '8.0.282'
java-version: '11'
distribution: 'corretto'
Copy link
Member Author

Choose a reason for hiding this comment

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

Same as build config for GraphQL Java

@@ -88,7 +83,7 @@ publishing {
asNode().children().last() + {
resolveStrategy = Closure.DELEGATE_FIRST
name 'graphql-java-extended-scalars'
description 'A library fo extended scalars for graphql-java'
description 'A library of extended scalars for graphql-java'
Copy link
Member

Choose a reason for hiding this comment

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

fo shizzle!

Copy link
Member Author

Choose a reason for hiding this comment

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

fo real!

Copy link
Member

@bbakerman bbakerman left a comment

Choose a reason for hiding this comment

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

merge away

@dondonz dondonz merged commit c03df43 into master Aug 21, 2023
@dondonz dondonz deleted the v21-update branch August 21, 2023 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants