Vampire
01/23/2023, 1:25 AMactions/setup-java
and with cache: gradle
?
The consistency check job just calls the Kotlin file and calls git
afterwards.
I don't see where it might need Java or Gradle.Piotr Krzemiński
01/23/2023, 8:16 AMactions/setup-java
, I want to use newest-ish JRE (LTS, to not make changes in consistency jobs too often), otherwise the default Java 11 is used (related bug and fix after this change). Java itself is still needed as runtime for Kotlin scripts.
About Gradle caching, it’s probably a copy-paste error - you’re right it seems not needed. Not sure how much it influences the running time, but I’ll remove it for the sake of simplicity.Vampire
01/23/2023, 8:34 AMJavaExec
Gradle task to 17 after you requiring it.
I don't really understand why you do as long as you don't use some 17+ lib, though. Kotlin is Kotlin is Kotlin. The bytecode version shouldn't matter much for the lib, should it? The probably only effect would be, that you force consumers to use 17+ while there is absolutely no need to, while at the same time making the consistency check a bit slower due to the need for installing Java 17.
May I ask what the reason was for that change?
Don't get me wrong, I second that move for any end project. I just don't like it for libs without reason and especially for one where the default in its typical usage scenario is 11. :-)Vampire
01/23/2023, 9:49 AMPiotr Krzemiński
01/23/2023, 9:50 AMVampire
01/23/2023, 10:15 AMPiotr Krzemiński
01/23/2023, 10:16 AMVampire
01/23/2023, 12:39 PMcache: gradle
even complained actually. 🙂