Skip to content

Conversation

@timtebeek
Copy link
Member

@timtebeek timtebeek commented Jan 17, 2026

Follows the pattern we already have for InlineMethodCalls, but now for Kotlin.

Follows the pattern we already have for `InlineMethodCalls`, but now for Kotlin.

Related openrewrite/rewrite-third-party#57
String identifier = identifierMatcher.group(1);

// Skip if already a placeholder or a keyword
if (identifier.equals("any") || identifier.equals("this") ||
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (identifier.equals("any") || identifier.equals("this") ||
if ("any".equals(identifier) || "this".equals(identifier) ||

String identifier = identifierMatcher.group(1);

// Skip if already a placeholder or a keyword
if (identifier.equals("any") || identifier.equals("this") ||
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if (identifier.equals("any") || identifier.equals("this") ||
if ("any".equals(identifier) || "this".equals(identifier) ||

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kotlin recipe Requested Recipe

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants