search

se-edu / addressbook-level3

:ab::three: Address Book sample application (Level 3)
https://se-education.org/addressbook-level3
MIT License
28 stars 415 forks source link

Upgrade to Java 17 #138

Closed fans2619 closed 1 year ago

fans2619 commented 2 years ago

Fixes #137

fans2619 commented 2 years ago

@damithc The app is runnable in Java 17 when simply changing the Java version in build.gradle and actually I don't see big problems.🤔 There are just some minor issues as pointed out by IDE and also in #122 . I'll look into those later.

image

damithc commented 2 years ago

@fans2619 That's good news. We also want to make sure relevant guides at https://se-education.org/guides/ are compatible with Java 17. e.g., JUnit tutorial JavaFX tutorial

We also need to decide if we should transition the project to use Java modules

@se-edu/tech-team-level1 What do you guys think?

damithc commented 1 year ago

Putting this hold for now.