Skip to content

Update kotlin compiler options to use compileOptions block in benchmark module #431

Merged
skydoves merged 1 commit into
skydoves:mainfrom
machado001:fix-gradle-build
Jan 17, 2026
Merged

Update kotlin compiler options to use compileOptions block in benchmark module #431
skydoves merged 1 commit into
skydoves:mainfrom
machado001:fix-gradle-build

Conversation

@machado001
Copy link
Copy Markdown
Contributor

@machado001 machado001 commented Jan 17, 2026

Guidelines

After migrate from Kotlin 2.2.x to 2.3.x on #421, we are required to migrate from kotlinOptions block to compilerOptions block strictly, as described in the official migration guide.

Types of changes

Fix #430

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Preparing a pull request for review

Code formatted using ./gradlew spotlessApply
Verified that the project builds and syncs successfully after migration.

Summary by CodeRabbit

  • Chores
    • Updated build configuration to use modern Kotlin DSL syntax while maintaining Java 11 compatibility.

✏️ Tip: You can customize this high-level summary in your review settings.

@machado001 machado001 requested a review from skydoves as a code owner January 17, 2026 01:19
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 17, 2026

Walkthrough

The benchmark module's Gradle build configuration is updated to migrate from the deprecated kotlinOptions { jvmTarget = "11" } syntax to the type-safe Kotlin DSL kotlin { compilerOptions { jvmTarget = JvmTarget.JVM_11 } } pattern, with corresponding import addition.

Changes

Cohort / File(s) Summary
Gradle Kotlin DSL Migration
benchmark/build.gradle.kts
Replaces deprecated kotlinOptions with kotlin { compilerOptions } block; updates jvmTarget to use JvmTarget.JVM_11 enum; adds org.jetbrains.kotlin.gradle.dsl.JvmTarget import.

Possibly related issues

Possibly related PRs

Suggested reviewers

  • skydoves

Poem

🐰 A little hop, a Gradle skip,
Deprecated code? We'll flip that script!
From old to new, the syntax gleams,
Type-safe targets fuel our dreams.


🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: migrating from kotlinOptions to compilerOptions block in the benchmark module, which matches the file-level changes and PR objectives.
Description check ✅ Passed The description follows the template structure with a clear guidelines section, types of changes checkbox, and preparation steps. It provides context linking to the Kotlin migration guide and confirms proper formatting and verification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Owner

@skydoves skydoves left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks for your contribution :)

@skydoves skydoves merged commit ef72719 into skydoves:main Jan 17, 2026
1 check passed
@machado001 machado001 deleted the fix-gradle-build branch January 17, 2026 11:59
@coderabbitai coderabbitai Bot mentioned this pull request May 7, 2026
3 tasks
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.

Using 'jvmTarget: String' is an error. Please migrate to the compilerOptions DSL.

2 participants