Upgrade Java to 17 LTS version

[santakadev]

Upgrade skeleton to Java 17 LTS version

[santakadev]

I'd remove log4j libraries from the skeleton. Is there any reason to keep them?

[JavierCane]

Yup!

I would vote for maintaining it 😬👼

Reasons why:

• Used in the bast majority of Java projects, so highly probable to be needed just after cloning the skeleton
• Useful for Java ecosystem newcomers in order to do not miss logs from libraries you can be actually using which, because of not having a main logger, would not have a visible output for you
• Consistency with the rest of skeletons where yo have an easy way to log things out

[santakadev]

Thanks! ❤️

Those are good reasons for maintaining log4j, so let's keep them 🙃

I thought it might be some dependencies that we forgot to remove 😅

[santakadev]

Not sure it these variables are something that Gradle needs. If they are not needed, I'd remove them.

[JavierCane]

Based on Gradle docs:

sourceCompatibility: Java version compatibility to use when compiling Java source. Default value: version of the current JVM in use.
?
targetCompatibility: Java version to generate classes for. Default value: sourceCompatibility.

So, my votes would go for:

• Maintain sourceCompatibility as it explicitly states the Java version to use while compiling instead of delegating it implicitly. Why: We would like to throw an error otherwise making it worth it to mention that the actual environment does not match the expected one.
• Maintain targetCompatibility because it acquiring the sourceCompatibility value by default does not seems like an easy assumption (at least in my case). However, I do not have any other reason apart from personal opinions here, so feel free to delete it if you prefer 😊

[santakadev]

Future improvement: Linux, Windows and Docker setup.

(I don't want to put effort on this right now 😬)

[JavierCane]

Minor improvements 😊

Decide what you prefer and go ahead merging it 🙌🚀