openrewrite / rewrite

Automated mass refactoring of source code.
https://docs.openrewrite.org
Apache License 2.0
2.15k stars 321 forks source link

Drop `HasSourcePath` in favor of `FindSourceFiles` #3755

Closed timtebeek closed 10 months ago

timtebeek commented 10 months ago

What problem are you trying to solve?

Both recipes do the same in slightly different ways; rather than maintain both we want to only have one going forward. Logging here, but should also be updated downstream.

Describe the solution you'd like

HasSourcePath fully removed, here and downstream, where it can be replaced with FindSourceFiles.

Additional context

timtebeek commented 10 months ago

Turns out HasSourcePath supported {.md,.txt} syntax, which FindSourceFiles does not (yet) support. Worked around this in downstream projects where we saw test failures; did not spot this elsewhere.