openrewrite / rewrite-migrate-java

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

Replace `javax.security.auth.Policy` with `java.security.Policy` #495

Closed BhavanaPidapa closed 3 weeks ago

BhavanaPidapa commented 3 weeks ago

What's changed?

image

What's your motivation?

I used org.openrewrite.java.ChangeType for the recipe RemovedPolicy

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. 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

cjobinabo commented 3 weeks ago

I've gone ahead and commited the suggestion. Thanks for looking at this @timtebeek

timtebeek commented 3 weeks ago

Great! Then feel free to merge whenever suits you best. :)