openrewrite / rewrite-migrate-java

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

Remove `java.lang.Thread.destroy()` or `java.lang.Thread.stop(java.lang.Throwable)` methods #500

Closed BhavanaPidapa closed 2 weeks 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

BhavanaPidapa commented 2 weeks ago

Few Unwanted files got pushed in this PR. To avoid confusion, I have created a new PR and closing this PR.