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.

Sign up for GitHub

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

Jump to bottom

Java 17 compatibility #1496

Merged
merged 16 commits into from
Nov 8, 2022
Merged

Java 17 compatibility #1496

merged 16 commits into from
Nov 8, 2022

Conversation

namtruong
Copy link
Collaborator

PR Description

  • A few dependencies upgrade and code refactoring to make the project compatible with Java 17 (being the next LTS release)
  • The existing unix startScript for Tessera distribution is kept as-is

Fixed Issue(s)

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@namtruong
Upgrade version of gradle wrapper
0e3a535
@namtruong
Upgrade version of google-java-format
6f873a4
@namtruong
Upgrade version of spotless plugin
f15020a
@namtruong
Upgrade jacoco version
500c655
@namtruong
Upgrade mockito version, update deprecated code
a25a3e7
@namtruong
Update sourceCompatibility. Update switch statements to use arrow ope…
52da272 
…rators
@namtruong
Temporarily suppress CVE-2022-41853 as this task is being worked on b…
89f79e5 
…y another PR
@namtruong
Gradle's updated unixStartScript template seems to be problematic for…
568d7a3 
… Tessera - hence making a copy of the old version of the template and instruct application plugin to use it instead
@namtruong
Update version jffi
fe8c135
@namtruong
spotlessApply
3d4b6ea
@namtruong
Upgrade gradle wrapper version
00c183c
@namtruong
setup step - change version of java distribution to 17
7d04e50
@namtruong
builds - change version of java distribution to 17
9fefd84
@namtruong
quorum-acceptance-test perhaps not need to be upgraded yet
1f0b26b
@namtruong
Update java version in base docker image
abadb5b
@namtruong
Update README doc
a4ab0ed
@antonydenyer antonydenyer merged commit 0e9e3c6 into Consensys:master Nov 8, 2022
@Krish1979 Krish1979 added 22.10.0 dependencies Pull requests that update a dependency file 22.10.0 labels Nov 8, 2022
@namtruong namtruong deleted the java-17-compatibility branch November 12, 2022 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Krish1979 Krish1979 Krish1979 approved these changes

@chris-j-h chris-j-h Awaiting requested review from chris-j-h chris-j-h is a code owner

Assignees
No one assigned
Labels
22.10.0 dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@namtruong @Krish1979 @antonydenyer