Skip to content

apply org.openrewrite.java.migrate.JavaBestPractices#560

Closed
dsgrieve wants to merge 2 commits into
microsoft:mainfrom
dsgrieve:best-practices
Closed

apply org.openrewrite.java.migrate.JavaBestPractices#560
dsgrieve wants to merge 2 commits into
microsoft:mainfrom
dsgrieve:best-practices

Conversation

@dsgrieve

@dsgrieve dsgrieve commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Ran openrewrite JavaBestPractices, e.g.,

./mvnw -U org.openrewrite.maven:rewrite-maven-plugin:run --define rewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-migrate-java:RELEASE --define rewrite.activeRecipes=org.openrewrite.java.migrate.JavaBestPractices --define rewrite.exportDatatables=true

JavaBestPractices bumps the version to 25.
In UnifiedDiariazer, GC_CAUSE was ambiguous (both a Token and a SupportedFlags) so I removed the static import and qualified the SupportedFlags.

@dsgrieve

dsgrieve commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@karianna

karianna commented Jul 5, 2026

Copy link
Copy Markdown
Member

@dsgrieve This breaks the Java 11+ matrix build, are you able to adjust the PR to add Java 25 without taking away the others?

@dsgrieve

dsgrieve commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@dsgrieve This breaks the Java 11+ matrix build, are you able to adjust the PR to add Java 25 without taking away the others?
I totally forgot about the matrix build.

@dsgrieve dsgrieve closed this Jul 6, 2026
@dsgrieve
dsgrieve deleted the best-practices branch July 6, 2026 20:25
@dsgrieve
dsgrieve restored the best-practices branch July 6, 2026 20:30
@dsgrieve
dsgrieve deleted the best-practices branch July 7, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants