Skip to content

Utilize a child-first strategy for resource loading#6414

Merged
shanman190 merged 3 commits intomainfrom
bugfix/child-first-resource-loading
Jan 2, 2026
Merged

Utilize a child-first strategy for resource loading#6414
shanman190 merged 3 commits intomainfrom
bugfix/child-first-resource-loading

Conversation

@shanman190
Copy link
Copy Markdown
Contributor

What's changed?

Load resources from the child classloader first, then attempt the parent classloader.

What's your motivation?

For recipe jars, such as rewrite-gradle and others, we desire to load resources from the recipe jar itself.

Anyone you would like to review specifically?

@jkschneider or delegate

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

@shanman190 shanman190 self-assigned this Dec 10, 2025
@shanman190 shanman190 added the bug Something isn't working label Dec 10, 2025
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Dec 10, 2025
@timtebeek timtebeek requested a review from jkschneider January 2, 2026 19:31
@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite Jan 2, 2026
@shanman190 shanman190 merged commit 101e384 into main Jan 2, 2026
2 checks passed
@shanman190 shanman190 deleted the bugfix/child-first-resource-loading branch January 2, 2026 22:13
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants