Skip to content

feat: lint native-breaking backtick test names#14

Merged
rasros merged 2 commits into
mainfrom
feat/native-safe-test-names
Jun 14, 2026
Merged

feat: lint native-breaking backtick test names#14
rasros merged 2 commits into
mainfrom
feat/native-safe-test-names

Conversation

@rasros

@rasros rasros commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Add a checkNativeSafeTestNames verification task to the com.eignex.lint convention plugin. It scans **/*Test/kotlin/**/*.kt and fails with a file:line on any fun \…`whose name contains(, ), or #`.
  • Wire it into check.

Why

Kotlin/Native (and Android dex) reject characters in backtick identifiers that the JVM accepts, so a test name like foo (#389) compiles for jvm but fails the native compile with a cryptic error. This catches it early with an actionable message instead of deep in the build.

Testing

./gradlew compileKotlin passes. Consumers pick it up via check once they bump the plugin version.

@rasros rasros merged commit 22901b7 into main Jun 14, 2026
2 checks passed
@rasros rasros deleted the feat/native-safe-test-names branch June 14, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant