openrewrite / rewrite-migrate-java

OpenRewrite recipes for migrating to newer versions of Java.
Apache License 2.0
92 stars 65 forks source link

Remove `Thread.destroy()` and `Thread.stop(Throwable)` #503

Closed BhavanaPidapa closed 1 week ago

BhavanaPidapa commented 2 weeks ago

What's changed?

image

What's your motivation?

I used org.openrewrite.java.migrate.RemoveMethodInvocation for the recipe ThreadStopDestroy

Anyone you would like to review specifically?

@cjobinabo @timtebeek

Have you considered any alternatives or workarounds?

Since it was not possible to test the recipe completely. I created sample files, created a local plugin using ./gradlew publishToMavenLocal then ran it in the Java8 Sample App using mvn rewrite:dryRun

Attaching the rewrite.patch file rewrite.patch

Checklist