Skip to content

Inline JavaTemplate fields at call sites#844

Merged
timtebeek merged 1 commit intomainfrom
tim/inline-javaparser-fields
Mar 31, 2026
Merged

Inline JavaTemplate fields at call sites#844
timtebeek merged 1 commit intomainfrom
tim/inline-javaparser-fields

Conversation

@timtebeek
Copy link
Copy Markdown
Member

Summary

  • Replaced 14 JavaTemplate fields stored on visitor classes with inline JavaTemplate.builder(...).build().apply(...) chains at each call site across 8 recipes
  • This removes unnecessary visitor state, following the modern pattern of building templates inline rather than caching them as fields
  • Net reduction of 30 lines; no behavioral changes — all existing tests pass

Replace stored JavaTemplate fields with inline builder chains across 8
recipes, eliminating unnecessary visitor state.
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Mar 31, 2026
@timtebeek timtebeek merged commit 78f9ddc into main Mar 31, 2026
1 check passed
@timtebeek timtebeek deleted the tim/inline-javaparser-fields branch March 31, 2026 16:04
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant