Uh oh!
There was an error while loading. Please reload this page.
Remove runtime dependency on maven-artifact - #266
Conversation
30afdcb to
8b2d3deCompareThis was primarily being used to check the Java version to handle < Java 7 in a different way for date formatting, but since Java 8 is the minimum version at this point it is no longer necessary. Also ends up removing the transitive dependency on commons lang which had minimal usage but replaced with an internal StringUtils equivalent.
8b2d3de to
2be6801CompareWill note this change came in as part of this commit but the compiler target at that point was already |
ryanrupp
commented
Nov 22, 2024
I also confirmed the latest release (6.2.2) on maven central is indeed producing Java 8 output bytecode (in case there was another build system/config doing the actual publishing). |
ryanrupp
commented
Nov 22, 2024
I guess would maybe use this PR (or a follow up) to call out the minimum Java version requirement in the README |
This was primarily being used to check the Java version to handle < Java 7 in a different way for date formatting, but since Java 8 is the minimum version at this point it is no longer necessary. Also ends up removing the transitive dependency on commons lang which had minimal usage but replaced with an internal StringUtils equivalent.