chore: bump rewrite-recipe-bom and integrate IsBlankToJdk and DefaultIfBlankToJdk recipes#1681
Merged
jonesbusy merged 3 commits intojenkins-infra:mainfrom Apr 9, 2026
Conversation
Collaborator
|
Thanks! |
jonesbusy
approved these changes
Apr 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps rewrite-recipe-bom from 3.26.1 to 3.28.0, which resolves rewrite-apache from2.23.2 to 2.24.0
rewrite-apache 2.24.0 includes two new recipes I contributed upstream in openrewrite/rewrite-apache#122 and openrewrite/rewrite-apache#124:
Both are added to MigrateCommonsLangToJdkApi's recipeList alongside the existing IsNotEmptyToJdk recipe.
Testing done
The upstream recipes are tested in openrewrite/rewrite-apache#122 and openrewrite/rewrite-apache#124. No additional tests are required here as the recipe wiring follows the same pattern as the existing IsNotEmptyToJdk entry.
The BOM bump was verified. All dependency changes are minor/patch increments with no major version jumps.
Notes
Also bumps jackson.version from 2.17.3 to 2.21.2 to resolve a conflict introduced by the new rewrite-bom requiring Jackson 2.21.x . The duplicate jackson.version property (silently set to 2.18.3) was also removed.
No behavioural change, only core stable Jackson APIs are used in this project.
Submitter checklist