Remove `OrderImports` from `JavaxAnnotationsToSpotbugs` - Githubissues

openrewrite / rewrite-jenkins

OpenRewrite recipes to continuously modernize Jenkins plugins.

Apache License 2.0

9 stars 8 forks source link

Remove `OrderImports` from `JavaxAnnotationsToSpotbugs` #11

Closed sghill closed 1 year ago

sghill commented 1 year ago

What's changed?

org.openrewrite.java.OrderImports removed from org.openrewrite.jenkins.JavaxAnnotationsToSpotbugs recipe.

What's your motivation?

Creates too many unrelated changes. This is sometimes viewed as noise, adding friction to potential merges.

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

If there is a way to order imports only in already-changed files, that would be preferred.

Any additional context

Fixes #10.

Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've added the license header to any new files through ./gradlew licenseFormat
[x] I've used the IntelliJ auto-formatter on affected files
[x] I've updated the documentation (if applicable)