-
Notifications
You must be signed in to change notification settings - Fork 111
Add UpgradePluginsForJava25 for surefire, failsafe, compiler, and byte-buddy #981
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.
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
Conversation
…e-buddy
Closes moderneinc/customer-requests#1820
| .containsPattern("maven-compiler-plugin</artifactId>\\s*<version>3\\.15\\.") | ||
| .containsPattern("maven-surefire-plugin</artifactId>\\s*<version>3\\.") | ||
| .containsPattern("maven-failsafe-plugin</artifactId>\\s*<version>3\\.") | ||
| .actual()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should byte-buddy show up in the test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it's just a UpgradeDependencyVersion I think we have that sufficiently covered elsewhere. Thanks for the suggestions though!
Summary
UpgradePluginsForJava25recipe to upgrade Maven plugins and dependencies to Java 25 compatible versionsUpgradeToJava25recipeProblem
After migrating to Java 25, Maven plugins need to be at minimum compatible versions. The existing
UpgradeToJava25recipe handles code migrations and<release>25</release>but does not upgrade plugin versions, causing build failures.Solution
New
UpgradePluginsForJava25declarative recipe that upgrades:maven-compiler-pluginto3.15.x(3.14.1 has known Java 25 issues)maven-surefire-pluginto3.1.xmaven-failsafe-pluginto3.1.xbyte-buddyto1.17.x(ASM 9.8 for Java 25 class files)9.1(first version with Java 25 support)Follows the established pattern from
UpgradePluginsForJava17andUpgradePluginsForJava21.Test plan
Existing tests pass
New test verifying plugin version upgrades
recipes.csvregenerated and validatedFixes moderneinc/customer-requests#1819
Fixes moderneinc/customer-requests#1820