Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

org.openrewrite.jenkins.ModernizePlugin does not migrate javax annotations to SpotBugs annotations #41

Open
basil opened this issue Aug 22, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@basil
Copy link
Contributor

basil commented Aug 22, 2023

As of commit d528b8c, running org.openrewrite.jenkins.ModernizePlugin against e.g. jenkinsci/view-job-filters-plugin@e0513aa does not migrate the javax annotations in the plugin to SpotBugs annotations as expected.

@sghill
Copy link
Collaborator

sghill commented Aug 22, 2023

It's separate for now, though the config allows for easily running both.

There's a balance to strike - I felt a PR that modernized several distinct things at once was unlikely to get merged.

@basil
Copy link
Contributor Author

basil commented Aug 23, 2023

That certainly could be true for plugins that are far behind and require many different migrations to be completed in order to catch up. Unfortunately many of these migrations are interdependent, making it difficult to separate them into distinct PRs.

In the long term I would love to see the recipes in this repository applied on a regular basis to the Jenkins ecosystem, with the changes changes possibly even automatically merged (with the consent of the community, broadly defined). To eventually reach that goal we should start thinking about what the high-level interface to these recipes should be.

@timtebeek timtebeek added the enhancement New feature or request label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

3 participants