Skip to content

Conversation

@srmalkan
Copy link
Contributor

@srmalkan srmalkan commented Aug 7, 2023

Closes #96

@timtebeek timtebeek changed the title added ambiguous method reference validation check (#96) ReplaceLambdaWithMethodReference should not replace ambiguous references Aug 7, 2023
@timtebeek
Copy link
Member

Thanks a lot for taking this on @srmalkan ! I've added a small change in 9c7a75b to hopefully remain safe without breaking existing functionality. Really glad to see this item resolved!

@timtebeek timtebeek self-requested a review August 7, 2023 21:15
@timtebeek timtebeek added the bug Something isn't working label Aug 7, 2023
@timtebeek timtebeek merged commit 60c789d into openrewrite:main Aug 7, 2023
@srmalkan srmalkan deleted the issue/96 branch August 8, 2023 03:51
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.

ReplaceLambdaWithMethodReference leads to ambiguous reference

2 participants