Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Only run the M2ApiDetector on Kotlin files #107

Merged
merged 1 commit into from
Apr 19, 2023
Merged

Only run the M2ApiDetector on Kotlin files #107

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions compose-lint-checks/src/main/java/slack/lint/compose/M2ApiDetector.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import com.android.tools.lint.detector.api.Severity.ERROR
import com.android.tools.lint.detector.api.SourceCodeScanner
import com.android.tools.lint.detector.api.StringOption
import com.android.tools.lint.detector.api.TextFormat
import com.android.tools.lint.detector.api.isKotlin
import com.intellij.psi.PsiNamedElement
import org.jetbrains.uast.UCallExpression
import org.jetbrains.uast.UElement
Expand Down Expand Up @@ -60,8 +61,10 @@ constructor(private val allowList: StringSetLintOption = StringSetLintOption(ALL
UQualifiedReferenceExpression::class.java,
)

override fun createUastHandler(context: JavaContext) =
object : UElementHandler() {
override fun createUastHandler(context: JavaContext): UElementHandler? {
// Only applicable to Kotlin files
if (!isKotlin(context.uastFile?.lang)) return null
return object : UElementHandler() {
override fun visitCallExpression(node: UCallExpression) = checkNode(node)

override fun visitQualifiedReferenceExpression(node: UQualifiedReferenceExpression) =
Expand All @@ -81,4 +84,5 @@ constructor(private val allowList: StringSetLintOption = StringSetLintOption(ALL
}
}
}
}
}