Skip to content

Conversation

davidh44
Copy link
Contributor

@davidh44 davidh44 commented Sep 3, 2025

Motivation and Context

Bump OpenRewrite dependency to latest version 3.14.0

Modifications

  • Port changes from unmerged PR to bump to 3.6.1 Bump OpenRewrite to 3.6.1 #6072

  • Bump Maven integ tests to use rewrite-maven-plugin 6.17.0

  • Bump Gradle integ tests to use rewrite-gradle-plugin 7.15.0

  • Old JavaType.Method constructors were dropped, added null args for new constructor

  • ConstructorToFluent recipe no longer works with args of type List, refactored to new recipe S3PojoToV2

  • ChangeMethodName no longer works with chained methods for newMethodName, e.g., method1().method2

    • Refactored transforms in ChangeTransferManagerSimpleMethods to TransferManagerMethodsToV2
    • Refactored transforms in ChangeExceptionTypes to new recipe SdkExceptionToV2

Testing

All tests pass

@davidh44 davidh44 requested a review from a team as a code owner September 3, 2025 20:07
Copy link

sonarqubecloud bot commented Sep 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
26.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@davidh44 davidh44 merged commit 3a01289 into feature/master/migration-tool-ga-release Sep 5, 2025
29 of 30 checks passed
@davidh44 davidh44 deleted the hdavidh/bump-openrewrite-3.14.0 branch September 5, 2025 20:33
Copy link

github-actions bot commented Sep 5, 2025

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants