Make org.openrewrite.java.migrate.lombok.log.LogVisitor public to avoid class loader access issues when loading recipe

[bryceatmoderne]

What's changed?

Made org.openrewrite.java.migrate.lombok.log.LogVisitor public

What's your motivation?

Workaround the following when loading rewrite-devcenter recipes:

class org.openrewrite.java.migrate.lombok.log.UseLog$1 cannot access its superclass org.openrewrite.java.migrate.lombok.log.LogVisitor (org.openrewrite.java.migrate.lombok.log.UseLog$1 is in unnamed module of loader io.moderne.recipe.RecipeClassLoader @71f96dfb; org.openrewrite.java.migrate.lombok.log.LogVisitor is in unnamed module of loader 'app')

Anything in particular you'd like reviewers to focus on?

Alternative solutions

Anyone you would like to review specifically?

@pstreef

Have you considered any alternatives or workarounds?

No

Any additional context

Checklist

• I've added unit tests to cover both positive and negative cases
• I've read and applied the recipe conventions and best practices
• I've used the IntelliJ IDEA auto-formatter on affected files

[pstreef]

I was hoping we could make it work without this, but I'm also happy to make that a future us problem