Skip to content

Conversation

@JordonPhillips
Copy link
Member

Issue

n/a

Description

This updates gradle to 7.2, which will unblock building on JDK 17

Testing

CI, and a local build

Additional context

JDK17 is an LTS release that contains text blocks, which will dramatically simplify a lot of codegen. There's also some other goodies like pattern matching switch statements and local variable type inference (e.g. var foo = "" instead of String foo = "")

Since the generators aren't GA yet, we could upgrade. We'll have to wait a bit though until it's supported everywhere we need it.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov-commenter
Copy link

Codecov Report

Merging #2793 (b46abb0) into main (4e23736) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2793   +/-   ##
=======================================
  Coverage   60.17%   60.17%           
=======================================
  Files         548      548           
  Lines       28592    28592           
  Branches     7000     7000           
=======================================
  Hits        17204    17204           
  Misses      11388    11388           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e23736...b46abb0. Read the comment docs.

@JordonPhillips JordonPhillips merged commit 4986bab into aws:main Sep 16, 2021
@github-actions
Copy link

github-actions bot commented Oct 1, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants